Bug 1147688 - [RFE] ux: Add 'copy' button to the SSH key displayed when creating a new host
Summary: [RFE] ux: Add 'copy' button to the SSH key displayed when creating a new host
Keywords:
Status: CLOSED DEFERRED
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: Frontend.WebAdmin
Version: 3.4.0
Hardware: Unspecified
OS: Unspecified
low
medium
Target Milestone: ---
: ---
Assignee: bugs@ovirt.org
QA Contact: Lukas Svaty
URL:
Whiteboard:
Depends On: 1154556
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-09-29 21:04 UTC by Matt Reid
Modified: 2020-04-01 14:51 UTC (History)
6 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2020-04-01 14:47:38 UTC
oVirt Team: Infra
Embargoed:
oourfali: ovirt-future?
rule-engine: planning_ack?
rule-engine: devel_ack?
rule-engine: testing_ack?


Attachments (Terms of Use)
SSH Public Key text box (37.20 KB, image/png)
2014-09-29 21:05 UTC, Matt Reid
no flags Details
Everything expanded (47.07 KB, image/png)
2014-10-02 14:40 UTC, Matt Reid
no flags Details

Description Matt Reid 2014-09-29 21:04:36 UTC
Description of problem:
You can opt to use an SSH key as authentication for a new host, if you change the selection from Password to SSH Public Key, you get to see a 3 line text box, which you can then use to scroll through the 15 line SSH key if you so desire.

There's more space at the bottom of the dialog, so the SSH key could have a bit more space.

Additional info:

Comment 1 Matt Reid 2014-09-29 21:05:12 UTC
Created attachment 942494 [details]
SSH Public Key text box

Comment 2 Yaniv Bronhaim 2014-10-02 13:59:21 UTC
I don't see the free space that you refer to. This space is being used when you expend the advanced parameters option.

Eldan, any comments or requests for improvement? 

fmpov it looks alright and i intent to close as WONTFIX, unless you have specific suggestion for improvement

Comment 3 Matt Reid 2014-10-02 14:40:11 UTC
Created attachment 943389 [details]
Everything expanded

There are other examples throughout the UI of adding in options and elements when needed, such as checking the Use External Providers checkbox earlier in that dialog.

With everything expanded on the New Host dialog, there's still some room at the bottom. You hide the SSH key box until the radio button is selected, perhaps when the box appears, it simply has an additional 2 or 3 lines worth of vertical space. It would have to push the Advanced Parameters down a bit, but there's room to do so, and it would be easier to scan the SSH key.

Comment 4 Eldan Hildesheim 2014-10-06 11:55:52 UTC
The purpose of the field is to be able to:
1. Click the field
2. Select all -  (the key strings)
3. Copy the hash to another place.

If this is the case, adding 3-4 more lines won't help to do the job better.
But adding a small button: "copy" will help. It is something that is done in many applications and it is very useful.

Comment 5 Eldan Hildesheim 2014-10-06 11:56:56 UTC
The purpose of the field is to be able to:
1. Click the field
2. Select all -  (the key strings)
3. Copy the hash to another place.

If this is the case, adding 3-4 more lines won't help to do the job better.
But adding a small button: "copy" will help. It is something that is done in many applications and it is very useful.

Comment 6 Matt Reid 2014-10-15 15:48:02 UTC
If that's the purpose of displaying it, it would definitely be nice to have a Copy button and streamline that process a bit.

Comment 8 Michal Skrivanek 2020-03-18 15:46:31 UTC
This bug didn't get any attention for a while, we didn't have the capacity to make any progress. If you deeply care about it or want to work on it please assign/target accordingly

Comment 9 Michal Skrivanek 2020-03-18 15:51:17 UTC
This bug didn't get any attention for a while, we didn't have the capacity to make any progress. If you deeply care about it or want to work on it please assign/target accordingly

Comment 10 Michal Skrivanek 2020-04-01 14:47:38 UTC
ok, closing. Please reopen if still relevant/you want to work on it.

Comment 11 Michal Skrivanek 2020-04-01 14:51:07 UTC
ok, closing. Please reopen if still relevant/you want to work on it.


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