Fix index.css button-menu color overriding behavior

[viniciusdc]

Reference Issues or PRs

closes #427

What does this implement/fix?

Put a x in the boxes that apply

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds a feature)
• Breaking change (fix or feature that would cause existing features not to work as expected)
• Documentation Update
• Code style update (formatting, renaming)
• Refactoring (no functional changes, no API changes)
• Build related changes
• Other (please describe):

Testing

• Did you test the pull request locally?
• Did you add new tests?

Documentation

Access-centered content checklist

Text styling

• The content is written with plain language (where relevant).
• If there are headers, they use the proper header tags (with only one level-one header: H1 or # in markdown).
• All links describe where they link to (for example, check the Nebari website).
• This content adheres to the Nebari style guides.

Non-text content

• All content is represented as text (for example, images need alt text, and videos need captions or descriptive transcripts).
• If there are emojis, there are not more than three in a row.
• Don't use flashing GIFs or videos.
• If the content were to be read as plain text, it still makes sense, and no information is missing.

Any other comments?

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
jhub-apps	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Aug 1, 2024 5:20pm
jhub-apps-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Aug 1, 2024 5:20pm

[viniciusdc]

due to my extensions in my editor, it also expanded the CSS config in a more readable way, let me know if that needs to be restore to original.

[krassowski]

I think this file is not meant to be edited by hand. While many projects use compilation at release time, this one appears to be committing compiled static assets to git repo. In that case the change likely needs to be done elsewhere and the static assets need to be recompiled. I guess this is why my PR #355 was not merged for over a month? I now force-pushed with static assets recompiled.

[jbouder]

I believe you're going to want to make your change here:

jhub-apps/ui/src/components/navigation/navigation.css

Line 2 in 610e36c

color: #0F1015 !important;

.

Then you'll need to do an npm run build from the ui directory and commit all changes

[viniciusdc]

I believe you're going to want to make your change here:

jhub-apps/ui/src/components/navigation/navigation.css

Line 2 in 610e36c

color: #0F1015 !important;

.
Then you'll need to do an npm run build from the ui directory and commit all changes

Nice, thanks @jbouder -- It was already late, so I didn't even noticed that file was an artifact of the build process XD, thanks for pointing out the right one

[viniciusdc]

@jbouder @aktech this is now correctly addressed thanks, ready for another look

[jbouder]

Makes sense, and works based on a quick test.

[viniciusdc]

@aktech may we get this merged and a new release cut? this is a blocker on Nebari 2024.7.1 using the latest jhub-apps

[jbouder]

@aktech may we get this merged and a new release cut? this is a blocker on Nebari 2024.7.1 using the latest jhub-apps

@aktech , you’re up

[aktech]

I'll do another release, thanks for getting this in.