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

Version label was almost not visible #491

Merged
merged 4 commits into from
Oct 25, 2023

Conversation

tymmesyde
Copy link
Member

No description provided.

Copy link

@gaurovgiri gaurovgiri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Setting opacity to 0.3 will help distinguish the version info from the clickable buttons in settings which are of opacity 0.4.

src/routes/Settings/styles.less Outdated Show resolved Hide resolved
Copy link
Member

@kKaskak kKaskak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks good with opacity 0.3, but it might require display changes on mobile. As it's brighter than before and fixed with the Navbar. What if we just put it in the same place as before but not fixed, visible only when you scroll down the settings and add some margin after it, so there is enough spacing between the label and the Navbar.

@gaurovgiri
Copy link

@kKaskak Yeah, I think doing so might be better.

src/routes/Settings/styles.less Show resolved Hide resolved
src/routes/Settings/styles.less Show resolved Hide resolved
@kKaskak kKaskak self-requested a review October 25, 2023 12:55
@tymmesyde tymmesyde merged commit a752213 into development Oct 25, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants