I have submitted a Pull Request to fix a visual bug where the left navigation sidebar drawer (q-drawer) renders with a transparent background in Dark Theme. This causes underlying table contents (like in Entity Find) to overlap with the menu text upon horizontal scrolling.
The code changes are limited strictly to WebrootVue.qvt.css to add the required solid background color override for the drawer in dark mode. Kindly review it when time permits.
I can confirm this is a correct fix. However why did you choose #121212? I don’t see this color anywhere in our various themes. Shouldn’t we stick to our palette?
I initially chose #121212 after using a color picker to inspect the drawer background in the existing dark theme. Based on that, it appeared to be the closest match, so I used it for the fix.
After your comment, I also looked through Moqui’s color palette and noticed that the framework generally uses the standard black background consistently instead of #121212.
I’m happy to update the PR to use the theme’s existing color if that’s the preferred approach. I agree it’s better to stay consistent with the project’s palette rather than introducing a new color value.
Yeah my recommendation would be to go for consistency. Meaning keep that color, or else suggest to change it across the theme, instead of mixing and matching.
I’ve addressed your feedback and updated the background color from #121212 to black to keep it consistent with the existing dark theme behavior. The changes have been committed. Could you please take a look and let me know your feedback?
I ran into something similar with dark mode UIs where the component background was inherited instead of having its own color. The drawer looks fine in light theme but becomes confusing when the content behind it is visible. Changing the drawer background directly seems like the right fix here. Did you notice any other components needing the same adjustment?