Bug 1369315 - Wrong focus when clicking Add environment variable
Summary: Wrong focus when clicking Add environment variable
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Management Console
Version: unspecified
Hardware: Unspecified
OS: Unspecified
medium
low
Target Milestone: ---
: ---
Assignee: bpeterse
QA Contact: Yadan Pei
URL:
Whiteboard:
: 1448830 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-08-23 06:43 UTC by Yadan Pei
Modified: 2020-05-06 17:39 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2020-05-06 17:39:40 UTC
Target Upstream Version:


Attachments (Terms of Use)
WrongFocus (1.51 MB, image/gif)
2016-08-23 06:43 UTC, Yadan Pei
no flags Details
Still_wrong_when_clicking_Add-Environment-Variable-Using-a-Config-Map (729.72 KB, application/octet-stream)
2017-07-06 11:35 UTC, Xingxing Xia
no flags Details

Description Yadan Pei 2016-08-23 06:43:13 UTC
Description of problem:
When click "Add environment variable", it often focus on the previous row, reproduces mostly on Firefox48

Version-Release number of selected component (if applicable):
openshift v1.3.0-alpha.3+31f6add-dirty(commit 1c4709f)
kubernetes v1.3.0+507d3a7
etcd 2.3.0+git


How reproducible:
Often on Firefox

Steps to Reproduce:
1.Create app from image 
2.Go to Builds Environment tab
3.Click "Add environment variable"

Actual results:
3. After clicking "Add environment variable", it will focus on the previous row(refer to attachment)

Expected results:
3. Should focus on the new added blank row


Additional info:

Comment 1 Yadan Pei 2016-08-23 06:43:41 UTC
Created attachment 1193158 [details]
WrongFocus

Comment 3 openshift-github-bot 2016-08-23 21:05:43 UTC
Commit pushed to master at https://github.com/openshift/origin-web-console

https://github.com/openshift/origin-web-console/commit/2207f9a24edfd817966fc7fead7a9a46769acc01
Update kve, fix bug #1369315, tested in Chrome, FF, Safari, IE

Comment 4 Yadan Pei 2016-09-14 08:39:57 UTC
Found PR already merged in v3.3.0.30:

[root@PC ~]$ cd ~/github.com/openshift/ose
[root@PC ose]$ git log --pretty="%h %cd - %s" v3.3.0.30 | grep 'Bump' | head -n 5
efc0955 Thu Sep 1 11:58:10 2016 -0400 - Bump origin-web-console (7d59453)
929913a Tue Aug 30 17:56:10 2016 -0400 - Bump origin-web-console (39ffada)
1501117 Fri Aug 26 12:25:27 2016 -0400 - Bump origin-web-console (b0cc495)
0a43160 Thu Aug 25 15:21:14 2016 -0400 - Bump origin-web-console (98cd97c)
0f0228d Wed Aug 24 16:47:38 2016 -0400 - Bump origin-web-console (18a0d95)
[root@PC ose]$ cd ~/github.com/openshift/origin-web-console
[root@PC origin-web-console]$ git log --pretty="%h %cd - %s" 7d59453 | grep '2207f9a'
2207f9a Tue Aug 23 16:32:02 2016 -0400 - Update kve, fix bug #1369315, tested in Chrome, FF, Safari, IE
 

From above, the commit in comment 3 is included in v3.3.0.30. But the problem is still reproduced in today v3.3.0.30 test

Comment 6 bpeterse 2017-05-09 14:26:25 UTC
*** Bug 1448830 has been marked as a duplicate of this bug. ***

Comment 7 Steve Goodwin 2017-07-05 15:02:30 UTC
Is this still an exiting bug?

I'm not seeing it on FireFox.

Comment 8 Xingxing Xia 2017-07-06 11:32:36 UTC
> How reproducible:
> Often on Firefox
As comment 0 says, it was "Often". I reviewed the draft before the reporter submitted the bug, I can remember it was indeed "Often".
But now I only reproduced one time after many times try when I only click "Add Environment Variable" (when the bug was reported, it only has "Add Environment Variable")

However, now console has "Add Environment Variable Using a Config Map or Secret", if I click it, the wrong focus can be easily *ALWAYS* reproduced, see following attachment

Comment 9 Xingxing Xia 2017-07-06 11:35:23 UTC
Created attachment 1294921 [details]
Still_wrong_when_clicking_Add-Environment-Variable-Using-a-Config-Map

Comment 10 bpeterse 2017-08-23 18:23:33 UTC
A PR opened to improve sorting behavior may also help this issue:
https://github.com/openshift/origin-web-console/pull/1962

Comment 11 bpeterse 2017-08-29 15:33:27 UTC
PR merged:
https://github.com/openshift/origin-web-console/pull/1962

Xingxing, is it possible to evaluate this again & see if we have further reduced the often,always,rarely?

Comment 12 Yadan Pei 2018-01-24 02:05:44 UTC
Ben,

Clicking "Add Value" could always set correct focus.
Clicking "Add Value from Config Map or Secret" could set correct focus except for the first time. The first time clicking on "Add Value from Config Map or Secret" will set focus on the previous key-value-editor entry.

Could we fix the wrong focus when clicking the first time? Or should we leave it as it is?

Comment 13 Yadan Pei 2018-01-24 02:06:42 UTC
Checked on v3.9.0-alpha.3+b41d1da-127

Comment 14 Yadan Pei 2018-01-26 09:21:58 UTC
Hi Ben, could you help confirm comment 12?

Comment 15 bpeterse 2018-01-26 19:00:14 UTC
I was able to reproduce this and definitely think we can take a look at getting it addressed. Moving it back to assigned.

Comment 16 bpeterse 2018-01-26 19:31:10 UTC
Opened PR https://github.com/openshift/origin-web-console/pull/2726

Comment 17 bpeterse 2020-05-06 17:39:40 UTC
We are no longer addressing bugs for 3.x versions unless there is an important security related issue.


Note You need to log in before you can comment on or make changes to this bug.