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 724140 - documents-panel: fix bugs
documents-panel: fix bugs
Status: RESOLVED FIXED
Product: gedit
Classification: Applications
Component: general
git master
Other Linux
: Normal normal
: ---
Assigned To: Gedit maintainers
Gedit maintainers
Depends on:
Blocks:
 
 
Reported: 2014-02-11 18:58 UTC by sébastien lafargue
Modified: 2014-02-15 15:08 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
disconnect signals sooner on tabs (1.10 KB, patch)
2014-02-11 19:02 UTC, sébastien lafargue
none Details | Review
disconnect signals sooner on tabs (5.28 KB, patch)
2014-02-11 21:02 UTC, sébastien lafargue
committed Details | Review
documents-panel: make GroupRow unselectable (2.09 KB, patch)
2014-02-12 09:06 UTC, sébastien lafargue
committed Details | Review
focus on text view when creating new window from tab (720 bytes, patch)
2014-02-12 11:56 UTC, sébastien lafargue
committed Details | Review
documents-panel: changes for style (1.83 KB, patch)
2014-02-15 14:54 UTC, sébastien lafargue
committed Details | Review
adwaita : gedit: documents-panel style changes (2.81 KB, patch)
2014-02-15 14:54 UTC, sébastien lafargue
committed Details | Review

Description sébastien lafargue 2014-02-11 18:58:35 UTC
there are several bugs with the documents-panel changes
Comment 1 sébastien lafargue 2014-02-11 19:02:40 UTC
Created attachment 268824 [details] [review]
disconnect signals sooner on tabs
Comment 2 sébastien lafargue 2014-02-11 21:02:35 UTC
Created attachment 268846 [details] [review]
disconnect signals sooner on tabs
Comment 3 Paolo Borelli 2014-02-11 21:31:26 UTC
Review of attachment 268846 [details] [review]:

pushed

I am leaving the bug open since as far as I understood you have some more patches in the pipeline
Comment 4 Paolo Borelli 2014-02-11 21:34:05 UTC
just as a reminder:

 - one of the things to fix is to not make the group rows selectable
Comment 5 sébastien lafargue 2014-02-12 09:06:33 UTC
Created attachment 268889 [details] [review]
 documents-panel: make GroupRow unselectable
Comment 6 sébastien lafargue 2014-02-12 09:26:55 UTC
encountered remaining bug :

- when making a new window from a tab ( contextual menu on tab,
  move to a new window, no focus on the text view at first.

- when hovering a row, close button appear and the row height change.

- on some configuration, the first row flicker when hovering it.
Comment 7 sébastien lafargue 2014-02-12 11:56:04 UTC
Created attachment 268902 [details] [review]
focus on text view when creating new window from tab

probably not directly related but it is here.
Comment 8 Paolo Borelli 2014-02-13 20:58:49 UTC
Review of attachment 268889 [details] [review]:

Committed, though I wonder if the group row should also not prelight since they are not selectable
Comment 9 Paolo Borelli 2014-02-13 20:59:09 UTC
Review of attachment 268902 [details] [review]:

committed. thanks
Comment 10 sébastien lafargue 2014-02-14 14:12:45 UTC
For the flickering bug, it is confirmed that it's a gtk+ bug :

https://bugzilla.gnome.org/show_bug.cgi?id=698730
Comment 11 sébastien lafargue 2014-02-15 14:54:19 UTC
Created attachment 269199 [details] [review]
 documents-panel: changes for style
Comment 12 sébastien lafargue 2014-02-15 14:54:58 UTC
Created attachment 269200 [details] [review]
adwaita : gedit: documents-panel style changes
Comment 13 Paolo Borelli 2014-02-15 15:07:41 UTC
Review of attachment 269199 [details] [review]:

thanks!
Comment 14 Paolo Borelli 2014-02-15 15:07:50 UTC
Review of attachment 269200 [details] [review]:

thanks!