Opened 8 days ago

Closed 8 days 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:
Has a Patch: no Platform: All

Description (last modified by Janus)

Before

After

Attachments (2)

Before.png (83.2 KB) - added by Janus 8 days ago.
After.png (82.1 KB) - added by Janus 8 days ago.

Download all attachments as: .zip

Change History (10)

Changed 8 days ago by Janus

Attachment: Before.png added

Changed 8 days ago by Janus

Attachment: After.png added

comment:1 Changed 8 days ago by Janus

Description: modified (diff)

comment:2 Changed 8 days ago by 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?

comment:3 Changed 8 days ago by Janus

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

comment:4 Changed 8 days ago by Janus

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

comment:5 in reply to:  2 Changed 8 days ago by Janus

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?

comment:6 in reply to:  2 Changed 8 days ago by Janus

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 Changed 8 days ago by humdinger

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 Changed 8 days ago by waddlesplash

Resolution: fixed
Status: in-progressclosed

Patch merged

Note: See TracTickets for help on using tickets.