Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Recon dialog: suggested types should be clickable (missing <label> DOM element) #6328

Closed
wetneb opened this issue Jan 27, 2024 · 12 comments · Fixed by #6397
Closed

Recon dialog: suggested types should be clickable (missing <label> DOM element) #6328

wetneb opened this issue Jan 27, 2024 · 12 comments · Fixed by #6397
Assignees
Labels
Good First Issue Indicates issues suitable for newcomers to design or coding, providing a gentle introduction. Theme: accessibility Highlights issues related to making the application more accessible to users with disabilities (WCAG Type: Feature Request Identifies requests for new features or enhancements. These involve proposing new improvements.
Milestone

Comments

@wetneb
Copy link
Sponsor Member

wetneb commented Jan 27, 2024

In the recon dialog, the suggested types are not clickable: one needs to click the radio button next to them.
This is bad accessibility.

Proposed solution

Enclose the type name and id in a <label> element associated to the radio button, so that one can click on the text as well.

Alternatives considered

none

Additional context

image

@wetneb wetneb added Type: Feature Request Identifies requests for new features or enhancements. These involve proposing new improvements. Theme: accessibility Highlights issues related to making the application more accessible to users with disabilities (WCAG Good First Issue Indicates issues suitable for newcomers to design or coding, providing a gentle introduction. labels Jan 27, 2024
@chelseaqnguyen
Copy link

hi! can i work on this?

@Redeem-Grimm-Satoshi
Copy link
Contributor

@wetneb Is this issue still available? since @chelseaqnguyen haven't sent a PR yet

@wetneb
Copy link
Sponsor Member Author

wetneb commented Feb 22, 2024

Sure, go ahead!

@itxsoumya
Copy link

itxsoumya commented Feb 22, 2024

hey @wetneb could you plz assign this issue to me

@Redeem-Grimm-Satoshi
Copy link
Contributor

@wetneb I can't seem to find the directory where the items in the column are rendered on the Standard Service Panel as show below;

Reco

Did I miss something?

@wetneb
Copy link
Sponsor Member Author

wetneb commented Feb 22, 2024

Have you tried this? https://openrefine.org/docs/technical-reference/code-contributions#locating-the-relevant-part-of-the-code-to-change

If so, which strings did you search for?

@Redeem-Grimm-Satoshi
Copy link
Contributor

Redeem-Grimm-Satoshi commented Feb 22, 2024

Yes @wetneb, Thanks for the feedback!

Searched "Reconcile each cell to an entity of one of these types" and "core-recon/cell-type",
Which got me to standard-service-panel.js and standard-service-panel.html

Still didn't find I wanted.. Currently looking into it

@tfmorris
Copy link
Member

The list of items comes from the backend reconciliation service based on the data sent to it from the column being reconciled. The example above seems to be from the Wikidata reconciliation service.

@wetneb
Copy link
Sponsor Member Author

wetneb commented Feb 23, 2024

Nicely done @Redeem-Grimm-Satoshi, you found the correct file: standard-service-panel.js.
This is indeed where the suggested types are rendered. Perhaps search for "type" in that file?

@Redeem-Grimm-Satoshi
Copy link
Contributor

Yes @wetneb , located the 'type'

type

@Redeem-Grimm-Satoshi
Copy link
Contributor

@wetneb Sent in a PR

@Redeem-Grimm-Satoshi
Copy link
Contributor

How it works;

recon.mp4

@wetneb wetneb added this to the 3.9 milestone Mar 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Good First Issue Indicates issues suitable for newcomers to design or coding, providing a gentle introduction. Theme: accessibility Highlights issues related to making the application more accessible to users with disabilities (WCAG Type: Feature Request Identifies requests for new features or enhancements. These involve proposing new improvements.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants