Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

double click in addressbar maximizes/restores the window instead of selecting the address #4922

Closed
asadotzler opened this issue Oct 18, 2016 · 5 comments · Fixed by #9282
Closed

Comments

@asadotzler
Copy link

asadotzler commented Oct 18, 2016

Test plan

#9282 (comment)


Did you search for similar issues before submitting this one? Yes

Describe the issue you encountered: Sometimes when I place the mouse pointer in the bottom part of the addressbar and double-click to select the address, rather than selecting the address, the window is maximized/restored. This is because the pointer tip can be several pixels into the bottom of the addressbar and still targeting the toolbar below the addressbar.

Expected behavior: When the pointer is visibly inside the addressbar and clicked it should focus the addressbar or turn into an i-beam cursor for text selection.

  • Platform (Win7, 8, 10? macOS? Linux distro?): Windows 10
  • Brave Version: 0.12.5
  • Steps to reproduce:
    1. with the window maximized move the mouse pointer into the bottom 20% or so of the addressbar
    2. double click
    3. see the window restored
@bbondy bbondy added this to the 0.12.7dev milestone Oct 19, 2016
@bbondy
Copy link
Member

bbondy commented Oct 19, 2016

Thanks for the bug report!

@bsclifton this is windows only, probably related to the titlebar removal. Could you take?

@bsclifton
Copy link
Member

The fix for this was included in 0.12.6... but it ended up causing a drag regression: #5225

Re-opening issue as it will be broken again if/when #5240 is accepted

@bsclifton bsclifton reopened this Oct 28, 2016
@bsclifton bsclifton removed this from the 0.12.6dev milestone Oct 28, 2016
@bsclifton bsclifton removed their assignment Oct 28, 2016
@luixxiul
Copy link
Contributor

luixxiul commented Jun 4, 2017

@bsclifton has this issue been resolved?

@luixxiul luixxiul added the needs-info Another team member needs information from the PR/issue opener. label Jun 4, 2017
@bsclifton
Copy link
Member

@luixxiul I don't believe so, no. This was not hitbox related, but related to how the textbox is labelled with CSS (see above PR which broke it).

@bsclifton bsclifton removed the needs-info Another team member needs information from the PR/issue opener. label Jun 5, 2017
@luixxiul
Copy link
Contributor

luixxiul commented Jun 5, 2017

yes I got it, I think I found a way of solving this. I'm working on it.

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.

5 participants