-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
[#12668] Instructor's Student Records Page: Feedback session panel chevron barely visible #12673
[#12668] Instructor's Student Records Page: Feedback session panel chevron barely visible #12673
Conversation
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.
Looks good! 1 small nit
@@ -6,7 +6,7 @@ <h2>Records in feedback sessions</h2> | |||
<div class="card-header cursor-pointer" (click)="session.isCollapsed = !session.isCollapsed"> | |||
<strong class="text-break">Feedback Session: {{ session.feedbackSession.feedbackSessionName }}</strong> | |||
<div class="card-header-btn-toolbar"> | |||
<tm-panel-chevron [isExpanded]="!session.isCollapsed"></tm-panel-chevron> | |||
<tm-panel-chevron [isExpanded]="!session.isCollapsed" [chevronColor]="'black'"></tm-panel-chevron> |
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.
I think let's just go with chevronColor="black"
without the [] property binding, since we are just assigning a static value (this is also aligned with the instructor-session-result-question-view
component)
Updated the PR as per the review suggestions. I've changed the chevron color assignment to a static value ( |
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.
LGTM! Thanks for contributing to TEAMMATES!
…panel chevron barely visible TEAMMATES#12668 (TEAMMATES#12673) * Fix chevron color issue * Replace property binding with static value assignation
Fixes #12668
Outline of Solution
In this PR, I've updated the color of the
tm-panel-chevron
from white to black. This change aims to improve the visibility of the chevron on white backgrounds, based on the feedback and suggestions from the community.Changes made:
chevronColor
inPanelChevronComponent
to black.I've tested this change and believe it enhances the visibility of the chevron without impacting other design aspects. I'm open to any further suggestions or feedback.