-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
feat(tags): Adds border customizability to <Tag/> #70269
Conversation
static/app/components/badge/tag.tsx
Outdated
@@ -18,6 +18,7 @@ import type {Color} from 'sentry/utils/theme'; | |||
import theme from 'sentry/utils/theme'; | |||
|
|||
export interface TagProps extends React.HTMLAttributes<HTMLSpanElement> { | |||
borderStyle?: 'solid' | 'dashed' | 'dotted' | 'double' | 'none'; |
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.
There are more border styles than this, but I just added the 2d ones that I thought were relevant - feel free to correct me if there's a better way to do this
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.
Removed double because it doesn't appear to do anything.
Removed none because it just becomes text at that point
Is there a good way to write a test for this? I couldn't get |
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.
A note in the story could be useful https://sentry.sentry.io/stories/?name=app/components/badge/tag.stories.tsx
This PR adds a
borderStyle
prop to the<Tag/>
component. It is set tosolid
by default, since that is what was the only option before.This will be needed in upcoming changes to the Assignee dropdown trigger on the Issue details page, detailed here