Skip to content
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

stop using javascript to calculate navHeight, make it a static height instead #225

Open
johnsBeharry opened this issue Jan 13, 2023 · 5 comments
Labels
bug Something isn't working

Comments

@johnsBeharry
Copy link
Member

There's been a bug where the javascript calculation for the "--navHeight" css variable. This may not be necessary if we just set it as a static value of 72px and let css breakpoints do the rest.

@johnsBeharry johnsBeharry added this to the January Sprint #0 milestone Jan 13, 2023
@MTG2000
Copy link
Contributor

MTG2000 commented Jan 13, 2023

Could you tell me a little bit more about the problem that has been happening with you??

Cause there was really a problem with the nav height being not properly applied, but a few months back, I thought I fixed the problem by migrating some of the usage to CSS variables, & the problem wasn't happening with me anymore.

Has it been happening for you recently??
If so, could you tell me about the scenario for the bug.


What is the approach that you are suggesting that we use with CSS-only??

We are not calculating the nav height using JS btw, the nav will take whatever height it needs, & then using JS we are getting this height & storing it in the global state, & a few components are using this height to set some values in their styles.

(mainly all the sticky items that needs to know how much away from the top they should be to NOT overlap with the navbar)

So if we follow a CSS-only approach, how will these items position themselves properly without knowing the nav height value??

@johnsBeharry johnsBeharry added the bug Something isn't working label Jan 16, 2023
@johnsBeharry
Copy link
Member Author

Yeah so this is why I'm suggesting we just have a fix height for the various sizes of the nav bar. The navigation bar should not need to be resizable. So then we won't have to use JavaScript at all to calculate anything.

The problem happens in safari primarily. It makes the scrolling of the side columns difficult, because a lot of it gets cut off among other things.

@MTG2000
Copy link
Contributor

MTG2000 commented Jan 17, 2023

The problem happens in safari primarily.

Which Safari btw?
Mobile or Desktop?

@johnsBeharry
Copy link
Member Author

johnsBeharry commented Jan 17, 2023 via email

@MTG2000
Copy link
Contributor

MTG2000 commented Jan 18, 2023

Okay...
I borrowed a Mac from my friend & tried the website, but I wasn't able to re-produce the issue...

I think it would be quicker if you would just record a video where you show me the scenario that is happening with you.

Btw, just make sure that your safari browser version is relatively updated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants