Opened 2 months ago

Closed 2 months 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 2 months ago.
After.png (82.1 KB) - added by Janus 2 months ago.

Download all attachments as: .zip

Change History (10)

Changed 2 months ago by Janus

Attachment: Before.png added

Changed 2 months ago by Janus

Attachment: After.png added

comment:1 Changed 2 months ago by Janus

Description: modified (diff)

comment:2 Changed 2 months 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 2 months 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 2 months ago by Janus

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

comment:5 in reply to:  2 Changed 2 months 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 2 months 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 2 months 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 2 months ago by waddlesplash

Resolution: fixed
Status: in-progressclosed

Patch merged

Note: See TracTickets for help on using tickets.