GTG

Drop gray background in the column used for sorting

Bug #345808 reported by Raphaël Hertzog
4
Affects Status Importance Assigned to Milestone
GTG
Fix Released
Medium
Izidor Matušov

Bug Description

The gray background in the column used for sorting is in direct conflict with the colors coming from tags. The result is not nice and makes it difficult to identify task accurately.

In particular I use the gray color for a tag that I called status:inactive so, when I sort by title most of the lines end up gray even those that are not inactive !!

Changed in gtg:
importance: Undecided → High
milestone: none → 0.1.1
status: New → Confirmed
Revision history for this message
IC Raibow (icrbow) wrote :

Just don't change color for tag column if there any colored tags.

Revision history for this message
Lionel Dricot (ploum-deactivatedaccount) wrote :

wiz > I don't understand your suggestion.

Revision history for this message
Lionel Dricot (ploum-deactivatedaccount) wrote :

I defer this bug to 0.2 because it's not critical and nor Bertrand nor me has any idea on how to achieve this. We spent some hours on this without success.

Changed in gtg:
milestone: 0.1.1 → 0.2
Revision history for this message
Antons Rebguns (arebgun) wrote :

Could you post a screenshot to make it painfully clear for me :) ? No matter what color I choose for a tag it's still very clearly identifiable. Maybe it's theme dependent?

Changed in gtg:
milestone: 0.2 → 0.3
Revision history for this message
Diogo Ferreira (diogo-underdev) wrote :

I have a suggestion on how to solve this but I'm not sure if it's doable.

When picking the color for backgrounds with tags (GTG/taskbrowser/tasktree.py:111-114) if the result color is None or if bg_color_enable is False, then the background color is set to the default cell render color.

There is a slight problem though, this default cell color can only be obtained after receiving map-event.

Revision history for this message
Michael Budde (mbudde) wrote :

I've taken a screenshot of what I think Raphael is describing. If this is not the case, then I'm sorry! :)

Revision history for this message
Paul Natsuo Kishimoto (khaeru) wrote :

Something relevant: http://www.pygtk.org/docs/pygtk/class-gtktreeview.html#method-gtktreeview--set-rules-hint

In Michael's screenshot, the sort column has alternating colours for the cells in the last three rows, where the cell colour is not set explicitly. If the rules-hint was set to False, then *some* themes *might* not use alternating colours.

...anyway, I don't think this is what Raphael is asking for. To not highlight the sort column is not a built-in feature of gtk.TreeView. Two options would be:

1. Extend the cell renderer used in the task tree to check whether the cell being rendered is in the sort column. If it is, and if the task on that row has no colour, take some step to change the cell-background colour in that cell.
2. Choose (I don't know how) a cell-background to use for tasks that have no colour, and *always* use a cell-background.

I don't like the second option, because it means that users who choose not to use many coloured tags will not have the benefit of the built-in GTK alternating-rows colouring.

The first option seems like a lot of work, but I guess it is possible.

There are some other bugs related to major changes in the taskbrowser, like bug #340022 (group by due date)...maybe it would be better to tackle that first and then see if this is still a concern.

Revision history for this message
Bertrand Rousseau (bertrand-rousseau) wrote :

I agree about th epriority of this bug. And we should also check the GNOME HIG for that. If it's an accessibility issue, then it's also worth doing it. A better solution is maybe to adopt a the color generated from the tags, but applying a filter that darkens it for the sorting color, so the "highlitgh sorting column" behavior is kept coherent.

Changed in gtg:
importance: High → Medium
Changed in gtg:
milestone: 0.3 → 0.2.9
Revision history for this message
Lionel Dricot (ploum-deactivatedaccount) wrote :

Izidor > I saw your fix, please push it to the trunk :-)

Changed in gtg:
assignee: nobody → Izidor Matušov (izidor)
Revision history for this message
Izidor Matušov (izidor) wrote :

This is solved for Tags pane. I am not sure about the active task pane. I thing it was successfully solved in 0.2.4 => closing this bug as Fixed Committed.

Changed in gtg:
status: Confirmed → Fix Committed
Izidor Matušov (izidor)
Changed in gtg:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.