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 738525 - [PATCH] UI: Open in the directory of the current document
[PATCH] UI: Open in the directory of the current document
Status: RESOLVED FIXED
Product: gnome-latex
Classification: Other
Component: general
3.14.x
Other Linux
: Normal enhancement
: unspecified
Assigned To: LaTeXila maintainer(s)
LaTeXila maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2014-10-14 12:51 UTC by Matthieu Baerts
Modified: 2014-10-16 15:51 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
patch (1.42 KB, patch)
2014-10-14 21:05 UTC, Matthieu Baerts 		none Details | Review

Description Matthieu Baerts 2014-10-14 12:51:40 UTC 
Hello Séb,

I think that most of the time, when clicking on the Open button, the user will want to open a file which is located not so far from the current one.

For me, it's better to have this behaviour because if we want to switch to the Home folder, we can find a shortcut on the left panel.

This is why I'm proposing this patch :-)

Best regards,

Matt

PS: sorry, I was not able to test it because I'm not using GTK 3.14...
Comment 1 Sébastien Wilmet 2014-10-14 20:53:21 UTC 
I think you forgot to attach the patch.
Comment 2 Matthieu Baerts 2014-10-14 21:05:26 UTC 
Created attachment 288552 [details] [review]
patch

Strange... I added it when writing "This is why I'm proposing this patch"

But no problem, here it is! :-)
Comment 3 Matthieu Baerts 2014-10-14 21:06:26 UTC 
(Ok, I think it's just because I missed the description of the attachment)
Comment 4 Sébastien Wilmet 2014-10-16 14:01:40 UTC 
Pushed, after a small change:

https://git.gnome.org/browse/latexila/commit/?id=fa730a5a9ab22a21ef13053e1e7c4116e93f6528

Thanks!
Comment 5 Matthieu Baerts 2014-10-16 15:51:00 UTC 
Thank you for this merge and this change!