Skip to content

feat: Apply general styling, text colors, and input background to the…#784

Merged
northdpole merged 1 commit intoOWASP:mainfrom
manshusainishab:Visibility-fix
Mar 5, 2026
Merged

feat: Apply general styling, text colors, and input background to the…#784
northdpole merged 1 commit intoOWASP:mainfrom
manshusainishab:Visibility-fix

Conversation

@manshusainishab
Copy link
Contributor

fixes #783

… explorer page, with minor layout and selector refinements.
Screenshot 2026-03-05 at 3 40 31 AM

also fixes a minor scss fix which is developed by PR #775 because of which the css is not applied to the page on main branch and the explore page was rendering with a white background

Now it is following the original UI which is on the current developed version and also fixes the text visibility issue of that page which is currently live.

… explorer page, with minor layout and selector refinements.
@manshusainishab
Copy link
Contributor Author

@northdpole this is a priority PR as the hidden text version is currently deployed and it is live please if you can review this and publish a version to update it.
Thanks

@PRAteek-singHWY
Copy link
Contributor

PRAteek-singHWY commented Mar 5, 2026

Hi @manshusainishab, thanks for the fix here.

I reproduced the same blocking build error on current main / local run:

SassError: expected "}" in application/frontend/src/pages/Explorer/explorer.scss (around line 130).

This breaks webpack compile and blocks frontend validation.
The missing closing brace after #debug-toggle in this PR resolves that compile issue.

For context, issue #783 was triggered after earlier merged PR #767,

and the global page-level related to #783 ((where multiple pages appeared with an unintended dark/blue background after the overscroll patch) handling is already covered in PR #768.

Could we please keep the scope of this PR limited to the Sass compile fix above, since issue #783 and other fixes for rest of teh pages is already handled in PR #768.

Thanks again for the quick fix.

@northdpole please look into this, since this is live and high priority.

@northdpole northdpole merged commit f33510b into OWASP:main Mar 5, 2026
2 checks 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.

The text is not visible on Open CRE Explore page

3 participants