nextcloud-libraries / nextcloud-libraries/nextcloud-vue

Color picker does no longer accept shortened RGB values

Open
#6,109 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

1. to develop bug feature: colorpicker
Dominant language
Vue
Stars
246
Forks
99
Avg merge
1d 20h
Merged PRs (30d)
103

Description

The color picker component NcColorPicker does no longer accept colors in the shortened format (like #ABC for #AABBCC) as colors for the color palette. This due to the implementation in hexToRGB that assumes a format with 6 digits.

This should either be fixed or the documentation should clearly state that only full hex RGB values are allowed.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start in src/components/NcColorPicker/NcColorPicker.vue at the linked hexToRGB implementation around lines 400-404. Check how the color palette handles shortened hexadecimal values and review the component documentation for the stated accepted format. Done means the behavior or documentation clearly resolves the #ABC case.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
frontend
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.