Modify ↓

Opened 5 years ago

Closed 3 years ago

Last modified 3 years ago

#19013 closed defect (fixed)

Tagging presets: Multiselect's rows too narrow

Reported by: skyper Owned by: team
Priority: minor Milestone: 21.10
Component: Core Version:
Keywords: template_report tagging preset multiselect row Cc:

Description

screenshot

What steps will reproduce the problem?

  1. Have a preset with multiselect and row equal to amount of values
  2. Open the preset and have a look at the multiselect

What is the expected result?

No scrollbar in the multiselect

What happens instead?

Always a scrollbar

Please provide any additional information below. Attach a screenshot if possible.

  • Always need adding +1 to the value of row to get rid of the scrollbar.
  • Even visible with row="2".
  • Please, find example preset​ where the screenshot was taken of. 
Relative:URL: ^/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2020-03-30 07:18:28 +0200 (Mon, 30 Mar 2020)
Revision:16220
Build-Date:2020-03-30 23:25:33
URL:https://josm.openstreetmap.de/svn/trunk

Attachments (0)

Change History (9)

comment:1 by skyper, 5 years ago

Summary: Tagging presets: Multiselect rows too narrow → Tagging presets: Multiselect's rows too narrow

comment:2 by Klumbumbus, 5 years ago

In 16482/josm:

see #19013 - Add rows to multiselects with less than 7 values. Use value+1 as rows due to the display bug.

comment:3 by skyper, 3 years ago

Still present with r18246.

comment:4 by Don-vip, 3 years ago

Milestone: → 21.09

comment:5 by Don-vip, 3 years ago

Resolution: → fixed
Status: new → closed

In 18254/josm:

fix #21408 - fix #19013, fix #21385, fix #21404 - Fix MultiSelect issues (patch by marcello)

comment:6 by skyper, 3 years ago

Have to take a look, but we might want to revert r16482 and other multiselect with few values.

comment:7 by skyper, 3 years ago

Icon height needs to be taken into account, too, see my comment 5 on #21408.

comment:8 by Don-vip, 3 years ago

Milestone: 21.09 → 21.10

Milestone renamed

in reply to:  6 comment:9 by skyper, 3 years ago

Replying to skyper:

Have to take a look, but we might want to revert r16482 and other multiselect with few values.

Damn, I totally forgot ro remove the empty lines, see #21508.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment
E-mail address and name can be saved in the Preferences .
Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.