Skip to content

Improve answer UI in the availability selector#521

Merged
ThiefMaster merged 1 commit into
indico:masterfrom
renefs:answer-ui
Apr 24, 2026
Merged

Improve answer UI in the availability selector#521
ThiefMaster merged 1 commit into
indico:masterfrom
renefs:answer-ui

Conversation

@renefs

@renefs renefs commented Mar 17, 2026

Copy link
Copy Markdown
Contributor

Solves #430

Screenshot 2026-03-17 at 14 20 48 Screenshot 2026-03-17 at 14 20 24 Screenshot 2026-03-17 at 14 20 11 Screenshot 2026-03-17 at 14 20 01
  • Answer view
  • Summary view
  • Grid view

This PR aims to make more accesible the slot selector, displaying specific icons and texts for each kind of response.

@renefs

renefs commented Mar 17, 2026

Copy link
Copy Markdown
Contributor Author

Look & feel should be consistent now in all the views

@renefs renefs changed the title WIP: Improve answer UI in the availability selector Improve answer UI in the availability selector Mar 17, 2026
@marinaej marinaej self-requested a review April 15, 2026 14:46
@ThiefMaster

Copy link
Copy Markdown
Member

This looks really good, only the summary seems a bit hard to see - the previous one had a bigger yellow background so it had better contrast than the new one...

image image

@renefs

renefs commented Apr 23, 2026

Copy link
Copy Markdown
Contributor Author
Screenshot 2026-04-23 at 12 17 27

Yeah, the icon had better visibility before, specially with the grey bg.

@renefs

renefs commented Apr 23, 2026

Copy link
Copy Markdown
Contributor Author
Screenshot 2026-04-23 at 12 24 45

Better if we add the background again?

@ThiefMaster

Copy link
Copy Markdown
Member

I think it looks much better

- Add text explaining the selection
- Display more meaningful incons for each selection

- Add new translations for various languages in messages.po

- Use the CheckmarkParenIcon in the ParticipantTable and Grid

- Add support for 'ifneedbe' status in participant and answer grids

- Use solid border on unavailable slot

@marinaej marinaej left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@ThiefMaster ThiefMaster merged commit 6e8bf24 into indico:master Apr 24, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants