-
Notifications
You must be signed in to change notification settings - Fork 190
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix: timeline in programs page breaking #249
Conversation
@iamkryptonite @nandini45 please review the solution i approached for resolving the issue. |
@nlok5923 the current date is still being shown in the wrong place. Also you can use gh-pages to deploy instead of heroku. |
@iamkryptonite as today is 7th of march so it is showing: please correct me @iamkryptonite if am missing something. |
@nlok5923 if you scroll at the very beginning you will see this. 7th of March(todays date) is being shown in the month of January. It should be in March. And according the Fade and Marker component will be adjusted. |
@iamkryptonite please can you test the deployed site again i think the changes are addressed in latest commit: |
@nlok5923 the timeline is overlapping the program title |
Thanks @Rahulm2310 for pointing out the issue will resolve it in the latest commit. |
@Rahulm2310 i think you were testing it at custom resolution as it will definitely throws some distortion in custom resolution as here positioning has been made using @iamkryptonite can you suggest me some clue to bring dynamicity in positioning. |
@iamkryptonite please can you review the changes |
@nlok5923 Looks good just a few small changes:
|
@iamkryptonite the dates for events in timelines are actual ones i think we can't put dummy dates there. Also i still doubt how can we stop overlapping of event components with date in smaller resolution. the event component has fixed size so do i need to alter the size of event component or i need to do something else to make it dynamic . rest points will get address in latest commit. |
removed overlapping at smaller resolutions . test deployment here: https://hardcore-booth-457055.netlify.app/ |
Looks good! |
@annabauza can u please review this pull request. |
Approved require one more review |
I don't quite understand the change here. I don't understand the diagram 😳 @iamkryptonite if you approve this if you think it looks good I am happy to merge this. We just need a second approval |
@nlok5923 could you fix the merge conflicts, please? |
@isabelcosta i will open a new pr for this changes as i need to close this pr due to force push. |
Description
To resolve this issue currently i had changed the positioning value as previously the
date[0][1]
for both the programs are different which results in incorrect positioning of timeline now it has been resolved by consumig valuedate[2][1]
asdate[2][1]
is nearly same for both the programs.Fixes #246
Type of Change:
Delete irrelevant options.
Code/Quality Assurance Only
How Has This Been Tested?
deployment link: https://hardcore-booth-457055.netlify.app/
Checklist:
Code/Quality Assurance Only