Skip to content

1433: Improve collection screen - #1518

Open
LeandraH wants to merge 3 commits into
mainfrom
1433-collection
Open

1433: Improve collection screen#1518
LeandraH wants to merge 3 commits into
mainfrom
1433-collection

Conversation

@LeandraH

Copy link
Copy Markdown
Contributor

Short Description

Just getting #1506 from a fork into our repo

Resolved Issues

Fixes a part of #1433


- Use heart icons for the favorite button to match the Sammlung tab
- Add accessibility labels to the favorite button
- Explain purpose of own vocabulary on the collection overview
- Tell users how to add favorites / own words in the empty states
1433: Improve discoverability of the collection feature
@deliverino

deliverino Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

LLM Review (verdigado-think)

React/TypeScript Correctness

  • src/routes/user-vocabulary-list/components/ListEmptyContent.tsx: The transition from a implicit return to a function block to handle the hasNoWords logic is clean and correctly typed.

Accessibility

  • src/components/FavoriteButton.tsx: Good addition of accessibilityLabel based on the favorite state. This ensures screen-reader users know whether they are adding or removing an item.

Testing

  • src/routes/favorites/__tests__/FavoritesScreen.spec.tsx: New test file correctly covers both the empty state (verifying the new hints) and the populated state.
  • src/components/__tests__/FavoriteButton.spec.tsx and src/routes/user-vocabulary-list/__tests__/UserVocabularyListScreen.spec.tsx: Tests were properly updated to reflect the new labels and accessibility properties.

Commit Messages

  • The commits follow the project convention (<Issue key>: Message).
  • Commit 505d085a mentions using heart icons, but this change is not present in the final diff. However, commit 78906e5a explicitly states "Narrow scope to empty-state hints and a11y labels", which explains why the icon changes were removed. The final state is consistent with the refined scope.

General Observations

  • The PR effectively improves discoverability by providing textual hints in empty states, which is a positive UX improvement.
  • The release notes file is correctly provided.
  • No changes to navigation or sensitive data were detected.
  • No labels are required as these are user-facing enhancements.

@LeandraH
LeandraH enabled auto-merge August 31, 2026 13:57
@LeandraH LeandraH changed the title 1433 collection 1433: Improve collection screen Aug 31, 2026
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.

2 participants