-
Notifications
You must be signed in to change notification settings - Fork 0
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
TM-6053-Pub-Pos-Edit-UI #2987
TM-6053-Pub-Pos-Edit-UI #2987
Conversation
@@ -154,6 +154,7 @@ const PublishablePositionCard = ({ | |||
<div className="position-form--input"> | |||
<label htmlFor="publishable-position-statuses">Publishable Status</label> | |||
<select | |||
disabled={disableEditDetails} |
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.
so, the disabled checkbox for the Exclude button has a little no sign show on mouse hover + the disabled discoloration
vsbut, the publishable status looks the same if its disabled or not (this is in dark mode - light mode has a slight grey to the arrow.
vsdef not in the A/C's - but, it might be reasonable to make this distinction somewhat clear to the end user.
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.
add something to make the disabled drop down a little more clear -- this is fine otherwise
className="publishable-position-inputs" | ||
id="publishable-position-statuses" |
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.
more of a comment - but, why do these have classes & ID's if there isn't any corresponding CSS for them?
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.
Thanks Me! 👍
This is the UI portion of the publishable positions edit functionality.
Ticket