Skip to content

fix(cloud): improve dark mode readability by updating styles and clas…#208

Merged
DenizAltunkapan merged 2 commits intomainfrom
149-bug-markdown-content-not-readable-in-dark-mode-cloud-page
Mar 30, 2026
Merged

fix(cloud): improve dark mode readability by updating styles and clas…#208
DenizAltunkapan merged 2 commits intomainfrom
149-bug-markdown-content-not-readable-in-dark-mode-cloud-page

Conversation

@DenizAltunkapan
Copy link
Copy Markdown
Member

@DenizAltunkapan DenizAltunkapan commented Mar 30, 2026

  • Remove hover:bg-gray-100 from dropdown items in cloud.component.html
  • Drop !important from dropdown hover in cloud.component.scss
  • Drop !important from .cloud-row:hover in cloud.component.scss
  • Only apply cursor-pointer / hover:underline to breadcrumb when !last (moved into .cloud-link SCSS class)
  • Remove redundant min-h-screen from cloud.component.html:1
  • Code review and security check

@DenizAltunkapan DenizAltunkapan self-assigned this Mar 30, 2026
Copilot AI review requested due to automatic review settings March 30, 2026 18:44
@DenizAltunkapan DenizAltunkapan linked an issue Mar 30, 2026 that may be closed by this pull request
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Cloud Page styling to improve text/element contrast in Dark Mode by replacing hardcoded light-theme Tailwind colors with theme CSS variables, while aiming to preserve Light Mode appearance.

Changes:

  • Introduced Cloud-specific theme classes (cloud-btn, cloud-card, cloud-modal, etc.) backed by CSS variables.
  • Updated Cloud Page template to use the new theme classes for header, breadcrumbs, table, dropdown, and file editor modal.
  • Improved loading indicator color behavior by using border-current.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File Description
frontend/src/app/pages/cloud/cloud.component.scss Adds theme-variable-based component styles for containers, buttons, dropdowns, table rows, and modal inputs.
frontend/src/app/pages/cloud/cloud.component.html Replaces hardcoded Tailwind light colors with the new Cloud theme classes across the page and modal.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

…umb affordance, remove redundant min-h-screen

Agent-Logs-Url: https://github.com/Vault-Web/vault-web/sessions/afaef1a3-f34f-4b6e-9aea-d7732ffe104a

Co-authored-by: DenizAltunkapan <93663085+DenizAltunkapan@users.noreply.github.com>
@DenizAltunkapan DenizAltunkapan merged commit 41f87aa into main Mar 30, 2026
3 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.

[Bug]: Markdown Content Not Readable in Dark Mode (Cloud Page)

3 participants