Wrap KRadioButton groups in KRadioButtonGroup

[MisRob]

🌱 Are you new to the codebase? Welcome! Please see the contributing guidelines.

Summary

learningequality/kolibri-design-system#650 introduces a new Kolibri Design System component, KRadioButtonGroup that ensures expected keyboard navigation in Firefox #10491.

Now we need to update all places in Kolibri to use it.

References

• Two first places are already updated in Use KRadioButtonGroup in the language switcher modal and on the Device settings page #12325
• This will close Keyboard navigation inside radio button group (Firefox) #10491

Guidance

• Find all <KRadioButton>s and wrap them in <KRadioButtonGroup> as described in https://develop--kolibri-design-system.netlify.app/kradiobutton/.
• If the current Kolibri Design System release doesn't yet have KRadioButtonGroup available, you can use devserver-with-kds command to run Kolibri with your local KDS

Acceptance criteria

• There is no KRadioButton left in Kolibri that's not wrapped in KRadioButtonGroup
• There are no regressions in radio button functionality in any of the updated places

[elkielki]

Hi, can I get assigned to this issue?

[MisRob]

Hi @elkielki, thank you - yes. Please follow the guidance closely and ask here if anything comes up.

Also note that the newest Kolibri Design System that has <KRadioButtonGroup> available is not yet merged to Kolibri's develop and we currently have some issue with yarn linking these two repositories. I'd recommend that in your working branch, you change the two package.json's in the very same way as here #12616 and then run yarn install. Let us know if there are any issues.

[adityashibu]

Hi is the issue still open?

[elkielki]

@MisRob Feel free to reassign this issue to @adityashibu. I followed the 'Getting Started' documentation, but I'm stuck on the command nodeenv -p --node=18.19.0. For some reason, it's not installing npm.

[AllanOXDi]

Hi @elkielki I have unsigned you and assigned @adityashibu . Is there any error you are getting from the terminal that we can look at?

[adityashibu]

Hi @AllanOXDi, Just to make it clear, I need to wrap KRadioButton in KRadioButtonGroup right? So something along the lines of:

<KRadioButton
            v-for="language in languageCol"
            :key="language.id"
            ref="languageItem"
            v-model="selectedLanguage"
            :buttonValue="language.id"
            :label="language.lang_name"
            :title="language.english_name"
            class="language-name"
 />

would become:

<KRadioButtonGroup>
    <KRadioButton
                v-for="language in languageCol"
                :key="language.id"
                ref="languageItem"
                v-model="selectedLanguage"
                :buttonValue="language.id"
                :label="language.lang_name"
                :title="language.english_name"
                class="language-name"
     />
<KRadioButtonGroup />

Right?

[MisRob]

Hi @adityashibu, yes. I'd ask you to study the guidance and all the links in the issue description where you will find plenty of examples of simple use-cases as well as more complex ones, see what's already done, documentation for components you will work with, and understand how to test that it works as expected in Firefox.

As you navigate Kolibri and preview the places you're refactoring, would you always please make a note about how you navigated as a user to a given place, and include this in your pull request description? It will help our QA team to locate where to test. Doesn't need to be elaborate, just few steps to get to each of those places. Thanks a lot.

[MisRob]

@elkielki as @AllanOXDi mentioned, we're glad to help you debug your development server setup if you can give us some logs. Seems you're experiencing troubles with a node version managemen - many people have good experience with Volta.

[MisRob]

Note that #12325 was just merged so the two first places are addressed in the latest develop. cc @adityashibu

[adityashibu]

Sure I'll have a look at it :)

[MisRob]

Hi @adityashibu, are you working on this or planning to? Just to know if we should keep the assignment or not.

[iamshobhraj]

Hey @MisRob can you assign me this issue?

[MisRob]

Hi @iamshobhraj, yes, thank you

[Maku38]

Hello , is this issue still open....if yes can u assign me?

[bjester]

@Maku38 this issue is currently assigned.

You’re welcome to find a "help wanted" issue with no assignee

[harshmohite04]

/assign

[AllanOXDi]

Hey @iamshobhraj. Someone is already assigned to this issue. If you want to work on something else, check out these other issues labeled help wanted" issue with no assignee

[iamshobhraj]

@MisRob , I opened a pr for this issue

[AllanOXDi]

Hi @iamshobhraj,

Thank you for your contribution! Our team will review the PR and provide feedback soon.

[yashsaraswat2004]

Is this issue still open?

[AllanOXDi]

Hi @yashsaraswat2004 .It's already assigned. If you want to work on something else, check out these other issues labeled help wanted" issue with no assignee

[iamshobhraj]

@MisRob, this issue should have been resolved by PR #12751. Is there anything further that needs to be addressed?

[MisRob]

Hi @iamshobhraj, thanks for the notice! Yes this is now closed by #12751