After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 741314 - [regression] No border / shadow underneath items dragged onto listviews, treeviews and Nautilus' sidebar
[regression] No border / shadow underneath items dragged onto listviews, tree...
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Themes
3.14.x
Other Linux
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2014-12-09 22:25 UTC by Jean-François Fortin Tam
Modified: 2014-12-12 04:23 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Adwaita: Draw drag highlight around treeview rows (1.94 KB, patch)
2014-12-11 05:19 UTC, Matthias Clasen
committed Details | Review

Description Jean-François Fortin Tam 2014-12-09 22:25:07 UTC
When you drag a mail into a different folder in Evolution, or a file into Nautilus' sidebar, you don't get a shadow/border of the target anymore, which makes it very hard to "aim".

I know from experience (with various users whose computers I maintain) that this was a significant usability issue in GNOME 3.0 that was fixed sometime around 3.2 or 3.4 (IIRC). With 3.14, we're back to the same problem.
Comment 1 Jean-François Fortin Tam 2014-12-09 22:30:18 UTC
Oh, and this feature used to cover both inserting "onto" and "inbetween" items in a list.
Comment 2 Matthias Clasen 2014-12-11 05:14:07 UTC
I see drop target highlighting work, in both tree views (try tests/testtreeview, a black line is drawn), and in the places sidebar (whenever a drop is possible, we show a 'New bookmark' placeholder)
Comment 3 Matthias Clasen 2014-12-11 05:19:16 UTC
I see drop target highlighting work, in both tree views (try tests/testtreeview, a black line is drawn), and in the places sidebar (whenever a drop is possible, we show a 'New bookmark' placeholder)
Comment 4 Matthias Clasen 2014-12-11 05:19:38 UTC
Created attachment 292508 [details] [review]
Adwaita: Draw drag highlight around treeview rows

This got lost somewhere.
Comment 5 Matthias Clasen 2014-12-11 05:21:27 UTC
I guess you were missing the frame around the row for dropping into. We lost that somewhere along the way. The black line that I mentioned above is the indication for dropping between rows, and that was still working.
Comment 6 Matthias Clasen 2014-12-11 12:41:55 UTC
Attachment 292508 [details] pushed as 17be8c3 - Adwaita: Draw drag highlight around treeview rows
Comment 7 Jean-François Fortin Tam 2014-12-12 04:23:01 UTC
Ah yes. I see that in Liferea, the "inbetween rows" treeview drag indication works.

Quick check: from a purely theoretical perspective I looked at your patch and it seems that you set "1px solid black" for both gtk-contained.css and gtk-contained-dark.css.

Shouldn't it be "1px solid white" for gtk-contained-dark.css ?