A GPL palette without names specified for colors causes jumping in status bar

Bug #1743708 reported by wizzor
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Low
Patrick Storz

Bug Description

A color palette generated with tools like GPick do not always contain a color name.

Loading a color palette like this causes the color hover text to be broken into two rows, making the UI expand the status bar, causing the color bar above to jump around, making color selection difficult.

Steps to repeat:

Load the attached gpl-file to inkscape palettes directory
Select the palette from the arrow on the right hand side of the color palette
Hover over the non-black colors to see jumpy behavior, notice that the hover text is split to two rows
Hover over the black color (with the name specified), to see no jumping behavior

Suggestion to fix:
Do not print color name when no name is specified. Prevent non-space invisible characters and line breaks from printing.

Tags: color ui
Revision history for this message
wizzor (visa-parviainen) wrote :
Revision history for this message
Patrick Storz (ede123) wrote :

Confirmed. It also causes an empty tooltip.

I assume using the hex color code if no name is given would be the easiest solution (in fact many pallettes only contain the hex code as name anyway)

Changed in inkscape:
status: New → Confirmed
status: Confirmed → Triaged
importance: Undecided → Low
Revision history for this message
Patrick Storz (ede123) wrote :

Fixed in
  https://gitlab.com/inkscape/inkscape/commit/c33682b3abf953d449f78f6d16a91bc4781d8d3f

Inkscape will now use the hexadecimal color code for unnamed palette colors.

Changed in inkscape:
assignee: nobody → Eduard Braun (eduard-braun2)
status: Triaged → Fix Committed
milestone: none → 0.93
tags: added: backport-proposed color
Revision history for this message
Patrick Storz (ede123) wrote :
Changed in inkscape:
milestone: 0.93 → 0.92.3
tags: added: ui
removed: backport-proposed
Bryce Harrington (bryce)
Changed in inkscape:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.