Page MenuHomePhabricator

Apply `:focus-visible` in Card with link and Accordion
Closed, ResolvedPublic1 Estimated Story Points

Description

Background

In T370503 we've decided to use :focus-visible (make the focus visible just when navigating with the keyboard) in those Codex components navigating to other pages or sections. Since :focus-visible is already implemented in Link and Tabs, we need to apply the :focus-visible in the following remaining components:

  • Card with link. We could evaluate applying :focus-visible on Cards including navigation.
  • Accordion. It expands/collapses content within the page, so the :focus-visible approach could be applied as well in this component.

Acceptance criteria for Done

  • Implement :focus-visible in those Codex components navigating to other pages or sections
    • Card with link
    • Accordion

Event Timeline

Thank you for tagging this task with good first task for Wikimedia newcomers!

Newcomers often may not be aware of things that may seem obvious to seasoned contributors, so please take a moment to reflect on how this task might look to somebody who has never contributed to Wikimedia projects.

A good first task is a self-contained, non-controversial task with a clear approach. It should be well-described with pointers to help a completely new contributor, for example it should clearly pointed to the codebase URL and provide clear steps to help a contributor get setup for success. We've included some guidelines at https://phabricator.wikimedia.org/tag/good_first_task/ !

Thank you for helping us drive new contributions to our projects <3

Volker_E added a project: CSS.
Volker_E updated the task description. (Show Details)

@ChandraPratap25 Please feel free to provide a patch!

Volker_E renamed this task from Apply focus-visible in Card and Accordion to Apply `:focus-visible` in Card with link and Accordion.Sep 12 2024, 9:20 AM

Change #1093301 had a related patch set uploaded (by GauriGuptaa; author: GauriGuptaa):

[design/codex@main] T372748: Implement :focus-visible in Card and Accordion components

https://gerrit.wikimedia.org/r/1093301

Volker_E added a subscriber: GauriGupta.

Removing @ChandraPratap25 due to inactivity, I assume @GauriGupta is the Phab profile name connected to the patch author, correct? :)

Volker_E set the point value for this task to 1.Nov 21 2024, 9:20 PM
Volker_E moved this task from Needs Refinement to Up Next on the Design-System-Team board.

@GauriGupta Do you have time to finalize the work on this in the upcoming two weeks?

Hi @GauriGupta! Am I assuming right, that you've run out of time to continue working on this? If we won't hear back in the next 24 hours from you, I'd take the patch and finalize it with you tagged as co-author.

fyi, @GauriGupta told me in a personal email that they gonna will work on the patch next day(s).

Change #1093301 abandoned by GauriGuptaa:

[design/codex@main] T372748: Implement :focus-visible in Card and Accordion components

https://gerrit.wikimedia.org/r/1093301

Change #1093301 restored by VolkerE:

[design/codex@main] T372748: Implement :focus-visible in Card and Accordion components

https://gerrit.wikimedia.org/r/1093301

Change #1093301 abandoned by GauriGuptaa:

[design/codex@main] T372748: Implement :focus-visible in Card and Accordion components

https://gerrit.wikimedia.org/r/1093301

Change #1093301 restored by VolkerE:

[design/codex@main] T372748: Implement :focus-visible in Card and Accordion components

https://gerrit.wikimedia.org/r/1093301

Change #1093301 had a related patch set uploaded (by VolkerE; author: GauriGuptaa):

[design/codex@main] styles: Implement `:focus-visible` in Card and Accordion components

https://gerrit.wikimedia.org/r/1093301

Change #1093301 merged by jenkins-bot:

[design/codex@main] styles: Implement `:focus-visible` in Card and Accordion components

https://gerrit.wikimedia.org/r/1093301

Change #1117223 had a related patch set uploaded (by Anne Tomasevich; author: Anne Tomasevich):

[mediawiki/core@master] Update Codex from v1.20.0 to v1.20.1

https://gerrit.wikimedia.org/r/1117223

Test wiki created on Patch demo by ATomasevich (WMF) using patch(es) linked to this task:
http://patchdemo.wmcloud.org/wikis/9c22725d87/w/

Change #1117223 merged by jenkins-bot:

[mediawiki/core@master] Update Codex from v1.20.0 to v1.20.1

https://gerrit.wikimedia.org/r/1117223

Test wiki on Patch demo by ATomasevich (WMF) using patch(es) linked to this task was deleted:

http://patchdemo.wmcloud.org/wikis/9c22725d87/w/