Consolidate style for "Please register or sign in to reply"
What does this MR do and why?
In issue comment threads, the "Please register or sign in to reply" message has a bottom border that is not necessary. Some instances of the CSS class for this message also had manually applied styles that could be achieved instead by applying tailwind util classes.
References
- "sign in to..." link on work item activity look... (#537264 - closed)
- ✂️ UX Paper Cuts 18.0 → Package (#512695 - closed)
Screenshots or screen recordings
Note: the main thing fixed in this MR is the second row below. The rest of these screenshots are provided to show that these instances were not affected.
| Before | After |
|---|---|
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
How to set up and validate locally
- Check out this branch
- In an incognito window, visit the GDK
- Look at an issue with comment threads
- You will see that the "Please register or sign in to reply" message is styled in a way that matches other instances of this in the app. It does not have a bottom border.
- Look at a design detail
- You will see that the "Please register or sign in to start a new discussion" message is unchanged from prod
- Look at an MR with comment threads
- You will see that the "Please register or sign in to reply" message is unchanged from prod
- Look at an MR with diff comments
- You will see that the "Please register or sign in to reply" message is unchanged from prod
MR acceptance checklist
Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
Related to #537264 (closed)
Edited by Chad Lavimoniere











