mirror of
https://github.com/zed-industries/zed.git
synced 2026-07-09 16:00:35 +00:00
Include icon guidelines in PR templates and contribution docs (#58855)
Release Notes: - N/A
This commit is contained in:
parent
a1a66114e9
commit
7eda38207a
3 changed files with 7 additions and 6 deletions
2
.github/pull_request_template.md
vendored
2
.github/pull_request_template.md
vendored
|
|
@ -2,7 +2,7 @@ Self-Review Checklist:
|
|||
|
||||
- [ ] I've reviewed my own diff for quality, security, and reliability
|
||||
- [ ] Unsafe blocks (if any) have justifying comments
|
||||
- [ ] The content is consistent with the [UI/UX checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)
|
||||
- [ ] The content adheres to Zed's UI standards ([UX/UI](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist) and [icon](https://github.com/zed-industries/zed/blob/main/crates/icons/README.md) guidelines)
|
||||
- [ ] Tests cover the new/changed behavior
|
||||
- [ ] Performance impact has been considered and is acceptable
|
||||
|
||||
|
|
|
|||
|
|
@ -84,10 +84,10 @@ When your changes affect UI, consult this checklist:
|
|||
|
||||
- Do all keyboard shortcuts work as intended?
|
||||
- Are shortcuts discoverable (tooltips, menus, docs)?
|
||||
- Do all mouse actions work (drag, context menus, resizing, scrolling)?
|
||||
- Does the feature look great in light mode and dark mode?
|
||||
- Are hover states, focus rings, and active states clear and consistent?
|
||||
- Is it usable without a mouse (keyboard-only navigation)?
|
||||
- Do all mouse actions work (drag, context menus, resizing, scrolling)?
|
||||
- Does the feature look great in light and dark mode themes?
|
||||
- Are hover states and focus indicators clear and consistent?
|
||||
|
||||
**Responsiveness**
|
||||
|
||||
|
|
@ -100,7 +100,7 @@ When your changes affect UI, consult this checklist:
|
|||
|
||||
**Platform Consistency**
|
||||
|
||||
- Is the feature fully usable on Windows, Linux, and Mac?
|
||||
- Is the feature fully usable on Windows, Linux, and macOS?
|
||||
- Does it respect system-level settings (fonts, scaling, input methods)?
|
||||
|
||||
**Performance**
|
||||
|
|
@ -113,6 +113,7 @@ When your changes affect UI, consult this checklist:
|
|||
**Consistency**
|
||||
|
||||
- Does it match Zed’s design language (spacing, typography, icons)?
|
||||
- Make sure to visit [the icon design guidelines](https://github.com/zed-industries/zed/blob/main/crates/icons/README.md)
|
||||
- Are terminology, labels, and tone consistent with the rest of Zed?
|
||||
- Are interactions consistent (e.g., how tabs close, how modals dismiss, how errors show)?
|
||||
|
||||
|
|
|
|||
|
|
@ -26,4 +26,4 @@ To introduce a new icon, add the `.svg` file to the `assets/icon` directory and
|
|||
- SVG files in the assets folder follow a snake_case name format.
|
||||
- Icons in the `icons.rs` file follow the PascalCase name format.
|
||||
|
||||
Make sure to tag a member of Zed's design team so we can review and adjust any newly introduced icon.
|
||||
Make sure to tag a member of Zed's design team (@zed-industries/design) so we can review and adjust any newly introduced icon.
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue