RadioList style update

[bpierre]

Current version:

New version:

Figma reference: https://www.figma.com/file/bZ1j7omfhdMCbtFahP3nEJ9B/Client?node-id=0%3A1

Tasks

• Update the RadioList component in aragonUI. (https://github.com/aragon/aragon-ui/pull/769)
• Add AppBadge inside of the RadioList.

[iwaduarte]

Could add more information? Is the only difference between them the icon? What is the goal here?
Should I fork them the aragon-ui and edit the component add this props? Why is the issue in this repo then?

[sohkai]

@iwaduarte The main difference is the positioning of the radio against its label; currently it's center, in the new version it should be at the top.

There are two steps:

1. Fork and edit this component in aragonUI
2. Update this repo with the new version, and make sure nothing breaks. Updating the SignerPanel to have an AppBadge would be even better :).

[iwaduarte]

@sohkai I understand. Althoug I am afraid to put my efforts and time and not have another PR merged into the platform. I have seen that you guys have a very long list of PRs that are not merged.
I have also just recently PR a very simple issue and although it seems okay (issue approved) I have not heard of anyone for merging or anything like that.

Probably that is not your main job either but very unusual time frame indeed.

[sohkai]

Yes understood @iwaduarte, and thanks for the feedback. We are working on this on an organizational level so that we can:

1. Reduce the number of unmerged PRs
2. Decrease the time it takes for a PR to be merged

[stvtortora]

hi, I want to start contributing. Is anyone working on this issue?

[bpierre]

@stvtortora Thanks for offering your help! I don’t think anyone is on this issue, feel free to grab it. 🚀

[mauerv]

It appears that a PR was merged to solve this. Any reason it's still open? I'm looking for some issues to contribute.

[bpierre]

@mauerv Yes, we still need to add a AppBadge inside the RadioList. Do you want to work on it? 🙂

[ghost]

Hey, i want to fix the App badge but can't find the component

[sohkai]

Hey @etherean06, the mentioned AppBadge component would be added to the SignerPanel, but we're actually in the process of removing this side panel altogether with a different design that will alleviate the need to add an AppBadge here!

Since we're doing this in #1465, I'll close this issue here.