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

Widget focus problem #45

Open
if-kenn opened this issue Jan 12, 2018 · 5 comments
Open

Widget focus problem #45

if-kenn opened this issue Jan 12, 2018 · 5 comments

Comments

@if-kenn
Copy link

if-kenn commented Jan 12, 2018

It looks like you closed #44 suggesting that it had to do with it being a 3rd party extension, it also looks like you don't seem to be responding to comments on closed issues, I am making a new post here.

From the @andyg5000 who initially opened the issue:

The Harvest widget will periodically shift the focus in the browser to the "Stop timer" button automatically on the page. When you're typing a comment and it shifts the focus, it caused the timer to be stopped.

@adunkman I don't think you understand what @andyg5000 is saying, this is not support for a 3rd party extension, this is a problem with the fact that your iframe widget steals the focus after an amount of time so that it actually interrupts a users workflow. It happens exactly when the timer shows that 1 minute has past.

I have made a very minimal HTML/iframe setup and a screen cast in order for you to understand the issue, please note that there is no extension involved with this.

harvest-widget-focus-issue

min-harvest-timer.html.txt
(note that I had to make the file .txt in order to upload)

@if-kenn
Copy link
Author

if-kenn commented Feb 13, 2018

@adunkman can you please respond? For something that is suppose to help productivity and the entry of time keeping, this bug is seriously impacting the use for our engineering team. It even seems like it got worse recently as it immediately now takes the focus.

@andyg5000
Copy link

You can also reproduce the issue in Chrome without any custom code:

  1. Load https://platform.harvestapp.com/platform/timer
  2. Start a timer
  3. Remove focus from the "stop timer" button
  4. Wait and watch the "stop timer" button steal focus (~ 1 minute)

@braddunbar
Copy link
Contributor

Hey @if-kenn and @andyg5000! 👋

Sorry about the delay here. I'll take a look at getting that focus-stealing fixed up and get back to you shortly.

@braddunbar
Copy link
Contributor

Hey again! 👋

Looks like you can stop this with the closable=false query parameter. It will remove the cancel button and it won't take focus at all.

I'll always ask around internally about making this the default behavior. It seems like that would be desirable since the cancel button is only useful in a dialog/iframe.

Let me know if that doesn't work out!

@if-kenn
Copy link
Author

if-kenn commented May 15, 2018

@braddunbar thanks, it looks like the British version of the word was being used closeable instead of closable!

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

No branches or pull requests

3 participants