Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

[dark shell] Top of title bar not sensitive to double-click when maximized #5327

Closed
peterflynn opened this issue Sep 24, 2013 · 9 comments · Fixed by adobe/brackets-shell#352
Assignees
Milestone

Comments

@peterflynn
Copy link
Member

  1. Maximize Brackets window
  2. Move mouse all the way to top of screen
  3. Double-click

Result: nothing happens

Expected: window is un-maximized (same as if you double-click in the middle of the titlebar)

Other windows apps are sensitive all the way to the very top of the screen. This is important to get the Fitt's Law "infinite edges" effect -- you can throw your mouse vertically to the top without needing any fine positioning.

@ghost ghost assigned JeffryBooher Sep 24, 2013
@peterflynn
Copy link
Member Author

@JeffryBooher I hit this fairly often, so I'm flagging it a Medium... not sure if it should be required for this sprint or not, though.

@JeffryBooher
Copy link
Contributor

This is similar to #5269

@peterflynn
Copy link
Member Author

I think it's a little bit different -- #5269 is only an issue because of the lack of visual separation between menubar and titlebar, and other Windows apps aren't normally draggable from the menubar area. But double-click at the top of a maximized window is common behavior in all Windows apps.

@JeffryBooher
Copy link
Contributor

@peterflynn it will be fixed with the fix to #5269. It's similar because both issues require some re-work of the WM_NCHITTEST handler which I will do together...

@peterflynn
Copy link
Member Author

Cool, got it.

@JeffryBooher
Copy link
Contributor

Actually I think it may not have been fixed with that change after all. Nominating for Sprint 33.

@JeffryBooher
Copy link
Contributor

FBNC @peterflynn

@njx
Copy link
Contributor

njx commented Oct 29, 2013

@peterflynn - could you confirm this so we can close it out? Thanks.

@peterflynn
Copy link
Member Author

Confirmed fixed... sorry for the delay

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants