Space for Gloobus Preview

Bug #690864 reported by Danielle Foré
32
This bug affects 7 people
Affects Status Importance Assigned to Milestone
Marlin
Fix Released
Low
Unassigned

Bug Description

A missed feature from NE is pressing the space bar for Gloobus Preview :)

Changed in marlin:
status: New → Confirmed
importance: Undecided → Medium
Revision history for this message
ammonkey (am-monkeyd) wrote :

I don't want a hard coded binary path to gloobus in marlin code, because not everybody use it and there's many lightweitght applications arround.

We got 2 options:

- a g_setting_entry with the binary path to bind the space key. If the entry is empty, it would do the default action idem est gnome-open / exo-open %s. The gloobus-preview package could write the key in the post-install script for example.
Any other application could use the binding.
- registering the accelerator via an extension. An extension system could be developed or not, (i really don't know for the moment).

Revision history for this message
Danielle Foré (danrabbit) wrote :

I think maybe the setting key would be better since it would ensure two extensions aren't fighting over the same keybinding, the latter would simply overwrite the former.

Revision history for this message
Mathijs Henquet (mathijshenquet) wrote :

On the otherhand, overriding eachothers prefs causes user to be unable to pick behaviour. Are there many other applications like gloobus out there? Becouse if there isn't whats against hardcoding?

We could also implement something 'like' gloobus preview in the Context pane. We need some more uses for the context pane anyway.

Revision history for this message
Danielle Foré (danrabbit) wrote :

I don't really agree with the phrase "we need some more uses for the context pane". Remember do one thing and do it well? :p At this moment, the only functionality I would add to the context pane is a thumbnail. Any more would be overkill. Is Marlin a file browser or a file viewer?

ammonkey (am-monkeyd)
Changed in marlin:
importance: Medium → Low
Revision history for this message
ammonkey (am-monkeyd) wrote :

bump just added previewer support
for the moment u can set the previewer path via dconf-editor: apps/marlin/preferences/previewer-path

Changed in marlin:
status: Confirmed → Fix Committed
xapantu (xapantu)
Changed in marlin:
milestone: none → 0.1
Revision history for this message
Chascon (chascone) wrote :

I'm using Marlin 0.1 installed via http://ppa.launchpad.net/marlin-devs/marlin-daily/ubuntu (official source, I read somewhere) and there is absolutely no listing for apps/marlin/preferences/ in gconf.

Revision history for this message
Danielle Foré (danrabbit) wrote :

Dconf, not Gconf.

Revision history for this message
Astarothyk (ceum-0491) wrote :

Es incorrecto, la ubicación es: org/gnome/marlin/preferences/previewer-path

Revision history for this message
ammonkey (am-monkeyd) wrote :

status fixreleased

Revision history for this message
ammonkey (am-monkeyd) wrote :

 status fixreleased

On 15 April 2013 15:33, am monkeyd <email address hidden> wrote:

> status fixreleased
>

Changed in marlin:
status: Fix Committed → Fix Released
Revision history for this message
ammonkey (am-monkeyd) wrote :

 status fixreleased

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.