Opened 6 years ago

Closed 6 years ago

#14480 closed bug (fixed)

ColumnListView: horizontal scroolbar doesn't keep into account hidden columns

Reported by: Janus Owned by: Janus
Priority: normal Milestone: Unscheduled
Component: Kits/Interface Kit Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Platform: All

Description (last modified by Janus)

Before

After

Attachments (2)

Before.png (83.2 KB ) - added by Janus 6 years ago.
After.png (82.1 KB ) - added by Janus 6 years ago.

Download all attachments as: .zip

Change History (10)

by Janus, 6 years ago

Attachment: Before.png added

by Janus, 6 years ago

Attachment: After.png added

comment:1 by Janus, 6 years ago

Description: modified (diff)

comment:2 by humdinger, 6 years ago

Isn't that a feature? Why should the scrollbar indicate there's something outside the currently shown contents it can scroll to, when the column is actually hidden and nothing will be revealed when scrolled?

comment:3 by Janus, 6 years ago

Summary: ColumnListView: horizontal scroolbar doesn't keep into account hidden columnColumnListView: horizontal scroolbar doesn't keep into account hidden columns

comment:4 by Janus, 6 years ago

Owner: changed from nobody to Janus
Status: newin-progress

in reply to:  2 comment:5 by Janus, 6 years ago

Replying to humdinger:

Isn't that a feature? Why should the scrollbar indicate there's something outside the currently shown contents it can scroll to, when the column is actually hidden and nothing will be revealed when scrolled?

Are you saying the error is in the Tracker?

in reply to:  2 comment:6 by Janus, 6 years ago

Replying to humdinger:

Isn't that a feature? Why should the scrollbar indicate there's something outside the currently shown contents it can scroll to, when the column is actually hidden and nothing will be revealed when scrolled?

Are you reading the issue backward? After the patch the scrollbar is aligned to the visible columns like the Tracker (image After).

comment:7 by humdinger, 6 years ago

Indeed, I misunderstood. I assumed you reported a new bug after a recent change. Now that I've seen your patch at Gerrit, it's clear your "After" is after you fixed it...

comment:8 by waddlesplash, 6 years ago

Resolution: fixed
Status: in-progressclosed

Patch merged

Note: See TracTickets for help on using tickets.