Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix bug with highlighting text & drag region on macOS #412

Merged
merged 1 commit into from
Apr 4, 2017

Conversation

alexbrazier
Copy link
Contributor

@RocketChat/core

Closes #410

Fixes bug with selecting text by changing drag region show in images below.
Green means it is draggable (meaning you can't highlight text). If you remove the server list it just gets rid of that drag region, and the rest stays the same.

image

image

@engelgabriel engelgabriel merged commit dc3df50 into master Apr 4, 2017
@engelgabriel engelgabriel deleted the bugfix/drag-region branch April 4, 2017 02:05
@Nosferous
Copy link

Nosferous commented Apr 4, 2017

Another part of this issue is that if you have a scroll bar due to many channels/DMs/Etc. the scroll bar is not clickable as a scroll bar. This means you can't scroll if you have a standard 2-button mouse without scroll-wheel/scroll-touch funtionality.

See screen:
I have a grey scroll bar on the left. If I click and try to scroll it just drags the Rocket.Chat.Electron window around.
rocketchatscrollissue

Using Rocket.Chat.Electron: 2.6.1
Server: 0.54.2 [running onto Ubuntu 16.04 LTS]
My Client Machine: OSX 10.9.5

Sidenote- My Rocket.Chat.Electron app doesn't look exactly like the one in my screenshot. When using the app there is no grey title bar for the app window. But once I screenshot it- OSX is revealing it in the snap. (Just incase anyone questions if I'm actually on 2.6.1- I am. But OSX on 10.9.5 is revealing the title bar during built in screen shot process).

@rodrigok
Copy link
Member

What about use just the top area to drag?

@alexbrazier
Copy link
Contributor Author

@rodrigok I'm happy to change it to just the top bar. Hopefully it should be obvious enough for most people.

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

Successfully merging this pull request may close these issues.

4 participants