-
-
Notifications
You must be signed in to change notification settings - Fork 33
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
Design: Create Style Guide Presentation (Radio Buttons/Checkboxes) #1839
Comments
09/25/2024
Feedback received:
|
09/27/2024
Waiting on stakeholder feedback |
Current slide is approved by stakeholders, but we might need to add a multi-select dropdown from the filters to this list based on whatever's finalized |
@thetanmancan This issue has a 'waiting for stakeholder label' but the last comment you made is for NilakshiS. I see this issue is in the deck with an image with a horizontal scroll... so I am confused. |
Hi @thetanmancan I referenced an old image for the design, but I see a horizontal scrollbar on the dev site. It's a bit inconsistent, there's one on the admin notes filter, however I don't see any on address filters. I can remove the horizontal scrollbar, but I'm not sure how we're planning on handling longer text, or do we not expect data longer than two lines? Here's some examples from the dev site where the longer text wraps: Horizontal scrollbar on the admin notesThe checkbox size is the same here, but the wrapped text is middle aligned to the checkbox The checkbox is smaller here, |
@NilakshiS I tested Admin Notes filter and did not see a horizontal scroll bar, even when I zoomed in on my browser. However, how the overlay behaves right now might not be a relevant conversation because I think how we're dealing with text strings in filter overlays is going to be changed. At least for some columns, there will be a 25-character limit in the width of the filter overlays according to #1651 , after which an ellipsis will be added and the rest of the text will be cut off. I will ask John at today's meeting about whether that limit is meant to apply to all column filters. If it is, the horizontal scroll bar won't be relevant, but even if it isn't, I think the text will still end up wrapping like it is now, in which case the horizontal scroll bar still won't be relevant, but I will have to confirm with John. If we end up not having a horizontal scroll bar, we can remove it from the design system and I'll run that by the stakeholders in our next meeting. As for the vertical scroll bar, that's determined by the browser anyways, right? Or can our site dictate how that looks? |
This comment has been minimized.
This comment has been minimized.
Hi @NilakshiS. Due to the advent of #2043, it seems like horizontal scroll bars will not be necessary for lists of nested checkboxes, so I removed the horizontal scrollbar from our design system mockup. I also noticed that the mockup includes dropshadow on the list of checkboxes, but the inclusion of dropshadow is inconsistent with the other components being shown in the same section on the style guide, and the doesn't seem to be included anywhere in the implementation of the list of checkboxes on the Dev site. Therefore, I'm removing the dropshadow unless you tell me that it needs to be there for some reason. If you think the 25-character limit within lists of nested checkboxes is something we should include in the style guide, please let me know and I'll make an issue for it. Thanks. |
Hi @NilakshiS. I noticed while testing these styles in the Dev site that we use two different colors for each of the components listed: one upon hover and one upon selection. This highlights a number of problems with our style guide.
Could you please reconcile these issues in the design system? I'll move this issue back to In Progress in the meanwhile. Thank you. |
Hi @thetanmancan thank you for the updates and making those changes!
However, now that I rechecked these, I realized that there has been a mistake, I was under the assumption that these are ReactJS styles (which would look the same across browsers) but instead these are browser native styles. The design as it currently looks is how it looks on Chrome Browser running on windows OS. Radio Buttons and Checkbox on different browsers
|
Hi @NilakshiS.
|
To address the above comment:
|
@NilakshiS Now that we have clarified scope, please provide update
|
Dependency
Overview
The style guide currently does not mention any styling for radio buttons and check boxes. Add these to the Figma Design system and style guide presentation.
Details
The style guide does not explicitly address the styling for radio buttons, though the Example Modal Dialog on the "Dialog Boxes and Modals" page does implicitly show an example from the app.
However the example uses the "Navy Blue - Primary (
#002E6D
)" color for the radio buttons while the website uses the default browser styling for them. The default styling is used in several other UI elements such as checkboxes and dropdowns.So we need to decide if we should change the styling of these elements to keep it consistent with the colors of the design system or keep it browser default since it's the style users would already be familiar with.
Action Items
Resources
The text was updated successfully, but these errors were encountered: