platform icon indicating copy to clipboard operation
platform copied to clipboard

Published 20 hours ago • verified publisher icon harvesthq

Widget focus problem

Open if-kenn opened this issue 7 years ago • 5 comments

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 avatar Jan 12 '18 20:01 if-kenn

@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.

if-kenn avatar Feb 13 '18 16:02 if-kenn

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)

andyg5000 avatar Feb 13 '18 16:02 andyg5000

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 avatar Feb 13 '18 16:02 braddunbar

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!

braddunbar avatar Feb 13 '18 18:02 braddunbar

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

if-kenn avatar May 15 '18 23:05 if-kenn