UI Proposal: Replace Unselected and More with Favorites

[ethanstrominger]

Intention: Make it easier to remove and add items from the different sections, and change titles of sections to make it more obvious

• Change the title Unselected to Favorites.

• Change the title More to Current Repo Users

• In Co-author section, add two buttons next to each name

  • a heart to add a person to Favorites. Greyed out if already in favorites
  • an x to remove a co-author without adding to Favorites
  • add a search icon at the top which brings up the List dialog

• In Favorites (previously titled Unselected) section, add two buttons next to each name
  - a + to add a person to the Co-authors section without removing from favorites. Greyed out if already selected.
  - an x to remove a person from the Co-authors section

  • add a search icon at the top which brings up the List dialog

• In Repository Users section (previously titled More), add two buttons to each user:
  - a + to add a person to the Co-authors section. Greyed out if not applicable.
  - a heart to add a person to the favorites section. Greyed out if not applicable.

  • add a search icon at the top which brings up the List dialog

[rkotze]

Needs more information please.

[ethanstrominger]

@rkotze - Updated

[ethanstrominger]

@rkotze What do you think of the proposed UI?

[ethanstrominger]

You can assign to me - I won't be able to get to it right away, can start in 3-5 weeks.

[rkotze]

@ethanstrominger Not sure I completely follow your proposal.

What I think would be helpful is being clear on the problem you are solving, with a problem statement. Could you add it to the issue, you can edit the issue under the ellipsis on the right.

I do like the idea of changing the wording for "unselected" and "more author" lists but I would be concerned with merging them as they do represent different groups of data.

"Unselected" is a list of people a user typical pairs with and is essentially "global" making it accessible across multiple repos. So to me, it makes sense what you are suggesting to call it favourites and using the heart symbol. Also changing the plus to a heart in the "more authors" to add to favourites would be good. This could be a nice contained issue to make.

"More authors" is a list of contributors to that repo and I think it's worth making that distinction.

Let's start with being clear on what you're solving and we can take it from there.

[ethanstrominger]

@rkotze I modified the description to hopefully clarify both my intentions and the proposed UI changes.

[rkotze]

@ethanstrominger thanks for that. I see you have updated your second comment but could you move to your first comment.

In Co-author section, add two buttons next to each name

• a heart to add a person to Favorites. Greyed out if already in favorites
• an x to remove a co-author without adding to Favorites
• add a search icon at the top which brings up the List dialog

The way Git Mob works for co-authoring you need to be a "favourite" before hand. You won't need the favourite button here. The co-authoring section will typically be a small list so I also don't think there is a need for a search.

Regarding where a user shows in a list, I've stuck with they only appear once across all the list. To avoid confusion and duplication. Greying out might not be very clear depending on their theme and it's quite limited adjusting colours in vs code.

In list dialog

This might be worth breaking out into a new issue and keep the focus around the UI list. It might help as this would be your first ticket and all this might be quite a big task. Let be know your thoughts.

[ethanstrominger]

Okay - I will create a separate issue for the list. I changed proposal so that instead of greying out, you just don't see an icon if it is not applicable. The buttons hopefully make it more obvious how to take an action and what it does. Having a + on the Repository Users section makes adding to the Co-authors a single step. IMO you should be able to add people to the Co-authors list without adding them to favorites, which explains the buttons in the other two sections.

[rkotze]

Great thanks. I will put this into the "todo" list and you can work on it when you're ready.

I've designed this extension to be in sync with Git Mob CLI and that does not support a "temp" co-author option. There would need to be some thinking around how to approach that, also debate the value of making the change.

[ethanstrominger]

Hi @rkotze I don't know when I will be able to get this so unassigned myself. I would love to work on i, but I have other things in the pipeline. If anyone wants to pair with me on this, I would make time.