Bug 1148066
| Summary: | [BLOCKED] No tabbing in Custom Preview Dialogue | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [oVirt] ovirt-engine | Reporter: | Kevin Alon Goldblatt <kgoldbla> | ||||
| Component: | Frontend.WebAdmin | Assignee: | Daniel Erez <derez> | ||||
| Status: | CLOSED DEFERRED | QA Contact: | Raz Tamir <ratamir> | ||||
| Severity: | low | Docs Contact: | |||||
| Priority: | low | ||||||
| Version: | --- | CC: | amureini, bugs, ehildesh, kgoldbla, lsurette, rbalakri, Rhev-m-bugs, srevivo, tnisan, ykaul, ylavi | ||||
| Target Milestone: | --- | Keywords: | UserExperience | ||||
| Target Release: | --- | Flags: | rule-engine:
ovirt-4.2+
|
||||
| Hardware: | x86_64 | ||||||
| OS: | Unspecified | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | Doc Type: | Bug Fix | |||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2017-07-19 12:29:46 UTC | Type: | Bug | ||||
| Regression: | --- | Mount Type: | --- | ||||
| Documentation: | --- | CRM: | |||||
| Verified Versions: | Category: | --- | |||||
| oVirt Team: | Storage | RHEL 7.3 requirements from Atomic Host: | |||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||
| Embargoed: | |||||||
| Bug Depends On: | 1064240, 1149778 | ||||||
| Bug Blocks: | |||||||
| Attachments: |
|
||||||
|
Description
Kevin Alon Goldblatt
2014-09-30 15:49:30 UTC
Created attachment 942787 [details]
screenshots, server and vdsm logs
Added logs again as they were not saved before
Hi Kevin, We usually use tabbing for easy navigation between editable widget (e.g. text-boxes/select-boxes/etc). The custom preview dialog is composed from multiple (potentially numerous) check-boxes, so I'm not sure tabbing would be convenient in this case. Or have you meant tabbing between other widgets? Tabbing should be enabled here too. The first snapshot and its disks should be selected by default. Tabbing to allow changes to the selected disks and to navigate to the next snapshot. Then to tab on to the Cancel and OK buttons (In reply to Kevin Alon Goldblatt from comment #3) > Tabbing should be enabled here too. The first snapshot and its disks should > be selected by default. Tabbing to allow changes to the selected disks and > to navigate to the next snapshot. Then to tab on to the Cancel and OK buttons afaik, there's no other occurrence/support of such behavior in the UI regarding tables. As mentioned in comment #2, I'm not sure it would be convenient for check-boxes/radio-boxes in this specific dialog. @Einav/Eldan - what do you think? how should we handle tabbing here? I will let Eldan chime in here as well, but I tend to agree with Kevin - I think that allowing tabbing between the cells in the row, row by row, may be very convenient (very much like e.g. navigating between table cells within a Word document or similar). before moving forward with this particular request, I believe that we need the dialog tab index infrastructure in place (see bug 1064240). there is a chance that we will need further infrastructure in order to support tables, especially ones with dynamic number of rows (e.g. in this case, number of rows depends on number of disks within the VM). in this case, another BZ may be reported. thanks. Generally speaking - 'Tab' should move the focus from left to right, top to bottom. Do we have this infrastructure? - if so, all the ui should work accordingly. In this particular ui, I think you wouldn't work with keyboard/Tabs, nevertheless, Tab navigation is a good practice. This depends on an RFE that was pushed out to 4.0 - pushing this one too. Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release. According to the correspondence on bug 1149778, we won't have the required infra in the foreseeable future. Closing this bug. We should revisit if the decision there changes. |