Skip to content
This repository was archived by the owner on Jan 7, 2026. It is now read-only.

fixed review-notice css#39

Merged
alexsigaras merged 1 commit intodevelopfrom
update-review-notice-css
Apr 3, 2025
Merged

fixed review-notice css#39
alexsigaras merged 1 commit intodevelopfrom
update-review-notice-css

Conversation

@jmtang2018
Copy link
Copy Markdown
Contributor

@jmtang2018 jmtang2018 commented Apr 3, 2025

This pull request includes changes to the css/dashboard.css and src/dashboard.py files to modify the review notice styling and improve the layout.

Styling and layout improvements:

  • css/dashboard.css: Removed the .review-notice h6 styling block and adjusted the footer styling to include a fixed height of 50px.
  • src/dashboard.py: Simplified the review notice header by removing the <h6> tag and its associated inline styling.

@jmtang2018 jmtang2018 added the enhancement New feature or request label Apr 3, 2025
@jmtang2018 jmtang2018 requested a review from Copilot April 3, 2025 14:58
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 pull request updates the review notice displayed on the dashboard by modifying its HTML structure to adjust styling.

  • Removed the
    tag surrounding the review notice text.
  • Adjusted the content within the review notice container in src/dashboard.py.
Files not reviewed (1)
  • css/dashboard.css: Language not supported
Comments suppressed due to low confidence (1)

src/dashboard.py:25

  • Removing the
    tag may cause the review notice to lose its semantic heading role and associated styling. Consider wrapping the text in a heading tag or applying equivalent CSS styling to ensure consistent appearance.
This repository is under review for potential modification in compliance with Administration directives.

@alexsigaras alexsigaras self-requested a review April 3, 2025 15:31
Copy link
Copy Markdown
Member

@alexsigaras alexsigaras left a comment

Choose a reason for hiding this comment

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

🚀

@alexsigaras alexsigaras merged commit bd2c841 into develop Apr 3, 2025
@alexsigaras alexsigaras deleted the update-review-notice-css branch April 3, 2025 15:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants