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

Clicking the scrollbar closes the card on Chrome #1404

Closed
Logicbloke opened this Issue Dec 19, 2017 · 6 comments

Comments

Projects
None yet
4 participants
@Logicbloke
Copy link

Logicbloke commented Dec 19, 2017

Hello,

I'm running wekan 0.60 and chrome 63.0.3239.84 (Official Build) (32-bit), when I try to scroll down to add a comment (especially if you have a good chunk of text and a sizeable checklist) as soon as you click the scrollbar the card closes itself.

I had to disable the numlock and use the PageDown key on my keyboard to get there and comment.

Please fix.
Thanks!

@xet7

This comment has been minimized.

Copy link
Member

xet7 commented Dec 20, 2017

@Logicbloke

I tested this. This bug seems to happen only with Chrome. I don't remember having this bug before, has there been some update to Chrome that has broken this functionality?

There is no bug when I use Firefox.

@Logicbloke

This comment has been minimized.

Copy link
Author

Logicbloke commented Dec 20, 2017

@xet7

This comment has been minimized.

@Logicbloke

This comment has been minimized.

Copy link
Author

Logicbloke commented Dec 20, 2017

@xet7 xet7 added the Bug label Dec 20, 2017

@xet7 xet7 added this to Usability Bugs in Wekan Roadmap Dec 20, 2017

@xet7 xet7 moved this from Usability Bugs to In progress: by other contributors, if they have time. Order: Random, depends on each contributor. in Wekan Roadmap Feb 20, 2018

@xet7 xet7 moved this from In progress: by other contributors. Order: random. to In Progress: xet7 with E in Wekan Roadmap Mar 5, 2018

@xet7 xet7 added the Accessibility label Apr 5, 2018

@xet7 xet7 added this to the 2018-06-01 milestone May 6, 2018

@xet7 xet7 removed this from In Progress: xet7 with E in Wekan Roadmap May 6, 2018

@xet7 xet7 added this to 1) In progress: by xet7, all paid Open Source Wekan work and all unrelated work. Order: from top to bottom of cards. in Wekan Roadmap Jun 8, 2018

@xet7 xet7 removed this from 1) In progress: by xet7, all paid Open Source Wekan work and all unrelated work. Order: from top to bottom of cards. in Wekan Roadmap Jun 24, 2018

@hupptechnologies

This comment has been minimized.

Copy link
Contributor

hupptechnologies commented Dec 8, 2018

Working on it.

hupptechnologies added a commit to hupptechnologies/wekan that referenced this issue Dec 8, 2018

hupptechnologies added a commit to hupptechnologies/wekan that referenced this issue Dec 8, 2018

@xet7 xet7 closed this in #2050 Dec 8, 2018

xet7 added a commit that referenced this issue Dec 8, 2018

- Fix [Clicking the scrollbar closes the card on Chrome](#1404)
  by changing [mquandalle:perfect-scrollbar to malihu-jquery-custom-scrollbar](#2050).
  that works also when clicking scrollbar in Chrome. Also added back required packages that were removed in PR.

Thanks to hupptechnologies and xet7 !

Closes #1404

xet7 added a commit that referenced this issue Dec 8, 2018

- Fix [Clicking the scrollbar closes the card on Chrome](#1404)
  by changing [mquandalle:perfect-scrollbar to malihu-jquery-custom-scrollbar](#2050).
  that works also when clicking scrollbar in Chrome. Also added back required packages that were removed in PR.

Thanks to hupptechnologies and xet7 !
@chotaire

This comment has been minimized.

Copy link

chotaire commented Dec 9, 2018

This change has definitely made Wekan less beautiful. Previously, the scrollbar would only appear when hovering it (or when using the mouse wheel). Now all lanes with a number of tasks that require scrolling would always display a thick scrollbar next to them. Would it be possible to auto-hide the scrollbar when it is not used? Thanks to the design of Wekan we will not need a visible scrollbar to identify that there are more tasks available than shown on the screen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment