-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Add layout engine icons #6705
base: develop
Are you sure you want to change the base?
Add layout engine icons #6705
Conversation
"displayName": "Two column view", | ||
"iconName": "2-column-view", | ||
"tags": "presentation, arrangement, layout, two, grid, columns", | ||
"group": "interface", |
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.
not sure if "interface" is the most appropriate group here
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.
seems fine 👍🏽
renameBuild artifact links for this commit: documentation | landing | table | demoThis is an automated comment from the deploy-preview CircleCI job. |
}, | ||
{ | ||
"displayName": "Two column view", | ||
"iconName": "two-column-view", |
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.
I think it would be better if all these new icons started with a layout-
prefix so that they could be auto-completed / auto-suggested in a more intuitive way. Then I think you could also drop the -view
suffix. Something like this:
layout-two-columns
layout-two-rows
layout-three-columns
layout-three-rows
layout-bottom-row-two-tiles
...
Note that you'll also have to update all the .svg
file names after you edit "iconName"
fields in this JSON file.
"displayName": "Two column view", | ||
"iconName": "2-column-view", | ||
"tags": "presentation, arrangement, layout, two, grid, columns", | ||
"group": "interface", |
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.
seems fine 👍🏽
<path d="M1 0C0.447716 -2.41411e-08 2.42698e-07 0.447715 2.18557e-07 1L0 6C-2.41411e-08 6.55228 0.447715 7 1 7L15 7C15.5523 7 16 6.55229 16 6V1C16 0.447716 15.5523 6.36101e-07 15 6.1196e-07L1 0Z" fill="black"/> | ||
</g> | ||
<defs> | ||
<clipPath id="clip0_279_19905"> |
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.
these clip paths on many of the icons are causing rendering issues on the page (see docs preview), please remove all extraneous SVG markup from these files:
Add layout engine icons