Bug 1677282

Summary: Not able to enter proxy URL in setting page
Product: Red Hat Satellite Reporter: rakesh kumar <rakumar>
Component: SettingsAssignee: satellite6-bugs <satellite6-bugs>
Status: CLOSED ERRATA QA Contact: Omkar Khatavkar <okhatavk>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.5.0CC: apatel, asharvit, egolov, hyu, mhulan, rlavi
Target Milestone: 6.7.0Keywords: Triaged, UserExperience
Target Release: Unused   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: foreman-1.22.0-0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-04-14 13:23:49 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description rakesh kumar 2019-02-14 13:02:28 UTC
Description of problem: When trying to enter proxy URL in Administer -> Settings > General > HTTP(S) proxy, as soon as we enter "/ or ."  cursor moved to search field. 

Facing the same issue for all other fields as well in Administer -> Settings, unable to enter "/ or ."  in any field.


Version-Release number of selected component (if applicable):
6.5.0.15

How reproducible:

Always


Steps to Reproduce:
Try mentioning proxy URL (like: https://proxy.example.com) in Administer -> Settings > General > HTTP(S) proxy.

Actual results:
The cursor moved to search field


Expected results:
Should be able to mention the value.


Additional info:

Comment 1 Marek Hulan 2019-02-14 15:27:06 UTC
Ron, any thoughts on this? Should we activate shortcut only after some keyboard shortcut? Moving to web-ui component as this is not only related to settings I believe. This is probably easily disableable in the code. Also as a workaround, copy pasting value prepared outside of Satellite UI should probably work.

Comment 4 Marek Hulan 2019-02-18 09:24:24 UTC
Ron, I think shift + slash (or alt + slash) is much better. I encountered this myself yesterday and it's pretty annoying. Is it an easy fix?

Comment 5 Walden Raines 2019-02-18 19:42:17 UTC
(In reply to Marek Hulan from comment #4)
> Ron, I think shift + slash (or alt + slash) is much better. I encountered
> this myself yesterday and it's pretty annoying. Is it an easy fix?

I would imagine it's an easy fix.  Is this something we want to target for an upcoming release?

Comment 6 Ron Lavi 2019-02-18 20:00:31 UTC
After investigating it for a while,
shift + slash changes the slash into a question mark,
alt + any key doesn't appear to get caught by the events listeners
and ctrl + / has other meanings in some browser,

What i found as a good solution
is to determine where the event came from, the event.target.tagName
I've opened a PR with a fix to disable this functionality if the key was pressed inside an input.

Link to PR: https://github.com/theforeman/foreman/pull/6489
Link to ISSUE: https://projects.theforeman.org/issues/26076

Comment 7 Ron Lavi 2019-02-20 10:33:29 UTC
PR was merged :)

Comment 11 Bryan Kearney 2019-05-16 16:02:17 UTC
Moving this bug to POST for triage into Satellite 6 since the upstream issue https://projects.theforeman.org/issues/26076 has been resolved.

Comment 12 vijsingh 2019-11-24 03:19:04 UTC
ON_QA Verified

@Satellite 6.7.0 snap 3.0


Steps/Observation:

 - Able to enter '/' or '.' in Administer -> Settings > General > HTTP(S) proxy and other settings fields

Comment 16 errata-xmlrpc 2020-04-14 13:23:49 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2020:1454