Skip to content
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

Docs: fix Color Palette #3237

Closed
zvenigorodskaia opened this issue Mar 24, 2023 · 1 comment · Fixed by #3393 · May be fixed by #3275
Closed

Docs: fix Color Palette #3237

zvenigorodskaia opened this issue Mar 24, 2023 · 1 comment · Fixed by #3393 · May be fixed by #3275
Assignees
Labels
BUG Something isn't working implementation Requires implementation

Comments

@zvenigorodskaia
Copy link
Member

zvenigorodskaia commented Mar 24, 2023

Vuestic-ui version: 1.6.3

Steps to reproduce

  1. https://vuestic.dev/ui-elements/color-palette
  2. colors dropdown in the header
  3. https://vuestic.dev/en/styles/colors
  4. https://vuestic.dev/en/getting-started/kitchensink#color-palette-title

What is the expected behavior?
add elementBorder 1px inner border only for colours with contrast <1, white in this case.
all other colours shouldn’t have the border

image

What is the current behavior?
image
image
image
image

Other information
Mockup
https://www.figma.com/file/PflCWFZC3xz3YCt7uX3MpP/Documentation?node-id=104-2262&t=dqvY8pPywdnx8hzB-0

@zvenigorodskaia zvenigorodskaia added BUG Something isn't working team discussion Feedback wanted implementation Requires implementation and removed team discussion Feedback wanted labels Mar 24, 2023
@kushich kushich self-assigned this Mar 29, 2023
@asvae asvae assigned m0ksem and unassigned kushich May 2, 2023
@m0ksem
Copy link
Collaborator

m0ksem commented May 4, 2023

Docs issue is closed in #3393, because this example is reworked.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BUG Something isn't working implementation Requires implementation
Projects
Archived in project
Status: Todo
Development

Successfully merging a pull request may close this issue.

3 participants