-
Notifications
You must be signed in to change notification settings - Fork 189
chore: Left drawer visual updates #3815
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
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3815 +/- ##
=======================================
Coverage 97.10% 97.10%
=======================================
Files 840 840
Lines 24472 24472
Branches 8637 8636 -1
=======================================
Hits 23763 23763
- Misses 659 702 +43
+ Partials 50 7 -43 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
56e2ecf
to
ede2fb9
Compare
97ec42f
to
e1e1f8a
Compare
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 overall, but I still recommend to wait for @gethinwebster review, because this is now everyones' feature, not just our squad's
@@ -47,4 +47,5 @@ export const tokens: StyleDictionary.BordersDictionary = { | |||
borderWidthField: '1px', | |||
borderWidthPopover: '2px', | |||
borderWidthToken: '2px', | |||
borderAiPanelWidth: '2px', |
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.
Why did you move it here? Do we expect it to be themed or anything?
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.
No, I don't expect to reuse it anywhere. Removed
bed5fd6
to
036a603
Compare
@@ -951,7 +951,8 @@ export type ShadowsTokenName = | |||
| 'shadowSticky' | |||
| 'shadowStickyEmbedded' | |||
| 'shadowStickyColumnFirst' | |||
| 'shadowStickyColumnLast'; | |||
| 'shadowStickyColumnLast' | |||
| 'borderAiPanelWidth'; |
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.
need to remove this too
| 'borderAiPanelWidth'; |
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
01d8114
to
0d6f13e
Compare
Description
This PR includes visual updates for the left global panel in accordance with the design bug bash (q
MzUMAouvFuAa
)Related links, issue #, if available: n/a
How has this been tested?
Review checklist
The following items are to be evaluated by the author(s) and the reviewer(s).
Correctness
CONTRIBUTING.md
.CONTRIBUTING.md
.Security
checkSafeUrl
function.Testing
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.