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

With no icon, Top Temp text does not align with other text (class navbar-text) on navbar #52

Closed
ManuelMcLure opened this issue Aug 24, 2021 · 10 comments
Labels
Done Should be fixed in next released - will be tested when released

Comments

@ManuelMcLure
Copy link

This is an example with Navbar Clock and Navbar Temp enabled, Discorded theme in UI Customizer:
image
If I select "Align icons to the left" it looks like:
image
Both Navbar Clock and Navbar Temp just set the class to "navbar-text"
With icons and "Align icons to the left":
image

@LazeMSS
Copy link
Owner

LazeMSS commented Aug 24, 2021

Thanks - will be fixed soon.

@ManuelMcLure
Copy link
Author

ManuelMcLure commented Aug 25, 2021

I installed the develop branch and it does seem to fix the text alignment with no icons. It would be nice if it also aligned the text when you have icons on the left, but I can see how that would be harder to do.

@LazeMSS
Copy link
Owner

LazeMSS commented Aug 26, 2021

I installed the develop branch and it does seem to fix the text alignment with no icons. It would be nice if it also aligned the text when you have icons on the left, but I can see how that would be harder to do.

Do you mean when you icons on the left aligned or when there are other icons (from other plugins)?

@ManuelMcLure
Copy link
Author

When there are icons on the left aligned:
image

@LazeMSS
Copy link
Owner

LazeMSS commented Aug 26, 2021

Ahh you have an icon added... hmmm... maybe that can be fixed also.

@LazeMSS
Copy link
Owner

LazeMSS commented Aug 26, 2021

Try updating it is fixed now - but it doesn't look pretty if there are other items with graph background

@ManuelMcLure
Copy link
Author

Looks good now. I don't think I'll personally use the graph background at all so this is OK for me.

@LazeMSS
Copy link
Owner

LazeMSS commented Aug 26, 2021

Did a quick fix more for small screensizes :)

@ManuelMcLure
Copy link
Author

Latest works the same as the previous one for me, which I suppose is expected since I don't have a small screensize :)

@LazeMSS LazeMSS added the Done Should be fixed in next released - will be tested when released label Aug 29, 2021
@LazeMSS LazeMSS mentioned this issue Sep 7, 2021
@LazeMSS
Copy link
Owner

LazeMSS commented Sep 7, 2021

Fixed in 0.0.1.5

@LazeMSS LazeMSS closed this as completed Sep 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Done Should be fixed in next released - will be tested when released
Projects
None yet
Development

No branches or pull requests

2 participants