-
Notifications
You must be signed in to change notification settings - Fork 276
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Tutorial groups
: Fix the contrast in the tutorial group rows in the dark mode
#8494
Tutorial groups
: Fix the contrast in the tutorial group rows in the dark mode
#8494
Conversation
…rast Pull latest changes from develop
WalkthroughThe recent updates involve enhancing the visual representation and interactivity of the tutorial groups table in a web application. New styling rules have been added to differentiate between users who are tutors and those who are not. The HTML template now dynamically applies these styles based on the user's tutor status, improving the user experience by visually distinguishing user roles within tutorial groups. Changes
Possibly related issues
Recent Review DetailsConfiguration used: CodeRabbit UI Files selected for processing (1)
Files skipped from review as they are similar to previous changes (1)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Out of diff range and nitpick comments (1)
src/main/webapp/app/course/tutorial-groups/shared/tutorial-groups-table/tutorial-groups-table.component.html (1)
Line range hint
37-37
: The use of@if
for conditional rendering based onmifOfDifferentLanguages
is correctly implemented. However, there seems to be a typographical error in the variable namemifOfDifferentLanguages
. It should likely bemixOfDifferentLanguages
.- [mifOfDifferentLanguages]="mifOfDifferentLanguages" + [mixOfDifferentLanguages]="mixOfDifferentLanguages"
This is also a dark mode issue related to tutorial groups. Do you want to fix this in this PR as well? #6448 |
Yes, thanks for pointing it out. I can fix that as well. |
After today's dev meeting, I talked to Ramona about this issue and she thought that since it is not super urgent to fix this, I will fix and create another PR for this towards the end of this week. Is it okay for you? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Manually tested on TS4, works as expected
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good - good improvement 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maintainer approved
…rast Pull latest changes from develop
Checklist
General
Client
authorities
to all new routes and checked the course groups for displaying navigation elements (links, buttons).Motivation and Context
When the tutors attempt to display all the tutorial groups, they face an issue with poor readability of the links in the corresponding rows due to low contrast in dark mode. This problem arises because the rows are highlighted in a cyan/blue color, making the clickable links (the titles of the tutorials) barely visible.
Description
The same color and contrast used to highlight elements in the left sidebar are also applied to the rows of the tutorial groups for tutors (Discussed with Ramona). By doing this, we maintain consistency in the coloring scheme and avoid hardcoded colors in the client. Additionally, we have removed the hovering effect when users hover over a row, as the row itself is not clickable, which was confusing for users. We keep the hovering effect for the clickable link (title of the tutorial).
Steps for Testing
Prerequisites:
Testserver States
Note
These badges show the state of the test servers.
Green = Currently available, Red = Currently locked
Review Progress
Code Review
Manual Tests
Screenshots
Summary by CodeRabbit