Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 491572 - Provide a shortcut for the full screen mode
Summary: Provide a shortcut for the full screen mode
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.6   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 4.6 M7   Edit
Assignee: Patrik Suzzi CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 489087 489249
Blocks: 491785 492811
  Show dependency tree
 
Reported: 2016-04-13 04:49 EDT by Lars Vogel CLA
Modified: 2016-05-02 07:09 EDT (History)
12 users (show)

See Also:
Lars.Vogel: review+


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Lars Vogel CLA 2016-04-13 04:49:19 EDT
+++ This bug was initially created as a clone of Bug #489087 +++

We should find a good shortcut to toogle the full screen mode.
Comment 1 Gautier de SAINT MARTIN LACAZE CLA 2016-04-13 05:00:01 EDT
For summarize what are the previous rejected propositions : 

- F11 : already taken by JDT (and will not be released)
- Ctrl+F11 : already taken by JDT (and will not be released)
- Ctrl+ALT+X should be avoid. See See https://bugs.eclipse.org/bugs/show_bug.cgi?id=477451#c10
- Alt+Enter : already taken by Eclipse (project properties)


I suggest Alt+F11.
Comment 2 Lars Vogel CLA 2016-04-13 05:04:42 EDT
(In reply to Gautier de SAINT MARTIN LACAZE from comment #1)
> For summarize what are the previous rejected propositions : 
> 
> - F11 : already taken by JDT (and will not be released)
> - Ctrl+F11 : already taken by JDT (and will not be released)
> - Ctrl+ALT+X should be avoid. See See
> https://bugs.eclipse.org/bugs/show_bug.cgi?id=477451#c10
> - Alt+Enter : already taken by Eclipse (project properties)
> 
> 
> I suggest Alt+F11.

Already taken, please check the existing keybindings before suggesting something. Also Ctrl+Alt+<X> shortcuts cannot be used. See https://bugs.eclipse.org/bugs/show_bug.cgi?id=477451#c10
Comment 3 Lars Vogel CLA 2016-04-13 05:05:27 EDT
(In reply to Lars Vogel from comment #2)
> > I suggest Alt+F11.

Ah, sorry. Alt+F11 seems available.
Comment 4 Mickael Istria CLA 2016-04-13 06:56:55 EDT
+1 for Alt+F11.
Comment 5 Brian de Alwis CLA 2016-04-13 10:51:38 EDT
On OS X, the system standard is Cmd-Ctrl-F.
Comment 6 Mickael Istria CLA 2016-04-13 10:57:17 EDT
(In reply to Brian de Alwis from comment #5)
> On OS X, the system standard is Cmd-Ctrl-F.

We could add multiple binding for the same operation, to also provide the OSX binding.
Comment 7 Lars Vogel CLA 2016-04-13 11:08:15 EDT
(In reply to Mickael Istria from comment #6)
> (In reply to Brian de Alwis from comment #5)
> > On OS X, the system standard is Cmd-Ctrl-F.
> 
> We could add multiple binding for the same operation, to also provide the
> OSX binding.

Sounds like we have to, as cmd dies not exists for Linux and Win
Comment 8 Dani Megert CLA 2016-04-15 09:50:28 EDT
(In reply to Lars Vogel from comment #7)
> (In reply to Mickael Istria from comment #6)
> > (In reply to Brian de Alwis from comment #5)
> > > On OS X, the system standard is Cmd-Ctrl-F.
> > 
> > We could add multiple binding for the same operation, to also provide the
> > OSX binding.
> 
> Sounds like we have to, as cmd dies not exists for Linux and Win

Not in general. Different ones should be assigned depending on the OS.
Comment 9 Patrik Suzzi CLA 2016-04-18 03:13:02 EDT
Assigning to myself for investigation.
Comment 10 Eclipse Genie CLA 2016-04-18 04:40:48 EDT
New Gerrit change created: https://git.eclipse.org/r/70839
Comment 11 Patrik Suzzi CLA 2016-04-18 04:42:09 EDT
Please, review the associated change.
Comment 13 Lars Vogel CLA 2016-04-18 04:49:59 EDT
Thanks Patrik
Comment 14 Patrik Suzzi CLA 2016-04-18 11:37:58 EDT
According to Brian's comment on Change 70839 the binding should be fixed using a proper style -> M3+F11

Another question raised is: 
Given that M3+F11 is already used in Cocoa and Carbon, which Keyboard Shortcut shall we use for fullscreen?

Note: 
If we can agree I'm going to add the cocoa+carbon binding to the next change, that I'll push shortly. Otherwise, I'm going to open a follow-up bug to address this later.
Comment 15 Lars Vogel CLA 2016-04-18 11:40:29 EDT
(In reply to Patrik Suzzi from comment #14)
> According to Brian's comment on Change 70839 the binding should be fixed
> using a proper style -> M3+F11
> 
> Another question raised is: 
> Given that M3+F11 is already used in Cocoa and Carbon, which Keyboard
> Shortcut shall we use for fullscreen?

I think Mac has its own shortcut for using full screen. Can you add the M3+F11 only for Linux and Windows?
Comment 16 Brian de Alwis CLA 2016-04-18 11:57:41 EDT
(In reply to Lars Vogel from comment #15)
> I think Mac has its own shortcut for using full screen. Can you add the
> M3+F11 only for Linux and Windows?

Use the 'platform' attribute on the keybinding.
Comment 17 Eclipse Genie CLA 2016-04-18 11:58:11 EDT
New Gerrit change created: https://git.eclipse.org/r/70876
Comment 18 Patrik Suzzi CLA 2016-04-18 11:58:59 EDT
Please review the attached change.
Comment 20 Lars Vogel CLA 2016-04-18 14:25:40 EDT
Thanks Brian for finding issues and Patrik for fixing them. The shortcut is now specific for Linux and Window. Mac has already an existing shortcut for that.
Comment 21 Rüdiger Herrmann CLA 2016-04-19 11:39:10 EDT
After switching to full screen, a tooltip window with a message like "You have gone full screen ... <steps to go back> ..." is shown.
Wouldn't it be useful to add a hint for the key binding to this tooltip?
Comment 22 Lars Vogel CLA 2016-04-19 11:44:46 EDT
(In reply to Rüdiger Herrmann from comment #21)
> After switching to full screen, a tooltip window with a message like "You
> have gone full screen ... <steps to go back> ..." is shown.
> Wouldn't it be useful to add a hint for the key binding to this tooltip?

Yes, see Bug 491785.
Comment 23 Rüdiger Herrmann CLA 2016-04-28 04:01:31 EDT
The Extras for Eclipse plug-in uses the same key binding (Alt+F11) to open a Launch Dialog [1]. This dialog, similar to 'Open Type', lists all launch configurations and allows to start or edit the selected items.

I am aware that F11 is used in browsers to go full-screen and therefore Alt+F11 may have seemed an obvious choice. But historically, in Eclipse, 'F11-based' key bindings are used for launching things - think of F11 (Debug), Ctrl+F11 (Run), Shift+Ctrl+F11 (Coverage). And for this reason, I have chosen Alt+F11 for the launch dialog.

Therefore, I'm asking to use a different key binding for the full-screen command. Would Shift+F12 be a viable alternative?

[1] https://github.com/rherrmann/eclipse-extras#launch-configuration-selection-dialog
Comment 24 Lars Vogel CLA 2016-04-28 04:17:07 EDT
(In reply to Rüdiger Herrmann from comment #23)
> The Extras for Eclipse plug-in uses the same key binding (Alt+F11) to open a
> Launch Dialog [1]. This dialog, similar to 'Open Type', lists all launch
> configurations and allows to start or edit the selected items.
> 

> Therefore, I'm asking to use a different key binding for the full-screen
> command. Would Shift+F12 be a viable alternative?
> 
I personally think the browser analogy is a good argument to keep the existing shortcut. We also asked via Platform.ui mailing list for feedback a while ago, which lead to this shortcut. -1 for a change now, 
 we are about to release M7.
Comment 25 Rüdiger Herrmann CLA 2016-04-28 06:12:39 EDT
(In reply to Lars Vogel from comment #24)
> (In reply to Rüdiger Herrmann from comment #23)
> > The Extras for Eclipse plug-in uses the same key binding (Alt+F11) to open a
> > Launch Dialog [1]. This dialog, similar to 'Open Type', lists all launch
> > configurations and allows to start or edit the selected items.
> > 
> > Therefore, I'm asking to use a different key binding for the full-screen
> > command. Would Shift+F12 be a viable alternative?
> > 
> I personally think the browser analogy is a good argument to keep the
> existing shortcut. We also asked via Platform.ui mailing list for feedback a
> while ago, which lead to this shortcut. -1 for a change now, 
>  we are about to release M7.
Sorry for being late here, I didn't follow the Platform/UI mailing list.
Given that Eclipse isn't a browser, how does Akt+F11 to go full-screen fit into the picture if all other F11 bindings are used for launching?