#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:
Has a Patch: no Platform: All

Description (last modified by Janus)

Before

After

Attachments (2)

Before.png (83.2 KB ) - added by Janus 13 months ago.
After.png (82.1 KB ) - added by Janus 13 months ago.

Download all attachments as: .zip

Change History (10)

by Janus, 13 months ago

Attachment: Before.png added

by Janus, 13 months ago

Attachment: After.png added

comment:1 by Janus, 13 months ago

Description: modified (diff)

comment:2 by humdinger, 13 months 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, 13 months 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, 13 months ago

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

in reply to:  2 comment:5 by Janus, 13 months 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, 13 months 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, 13 months 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, 13 months ago

Resolution: fixed
Status: in-progressclosed

Patch merged

Note: See TracTickets for help on using tickets.