Skip to content

feat(records): add empty state indicator when no records found#47

Merged
ardevd merged 2 commits intomainfrom
no-records-indicator
Mar 29, 2026
Merged

feat(records): add empty state indicator when no records found#47
ardevd merged 2 commits intomainfrom
no-records-indicator

Conversation

@ardevd
Copy link
Copy Markdown
Owner

@ardevd ardevd commented Mar 29, 2026

No description provided.

@ardevd ardevd self-assigned this Mar 29, 2026
@ardevd ardevd added the enhancement New feature or request label Mar 29, 2026
Copilot AI review requested due to automatic review settings March 29, 2026 21:14
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds an “empty state” UI for the Records list screen so users see a clear message when a query returns no records.

Changes:

  • Added records_empty_list string resource (EN + NO).
  • Added an emptyText TextView to fragment_records_list.xml.
  • Updated RecordsListFragment to toggle between recycler view vs. empty message on success.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
app/src/main/res/values/strings.xml Adds default empty-state string.
app/src/main/res/values-no/strings.xml Adds Norwegian empty-state string.
app/src/main/res/layout/fragment_records_list.xml Adds an emptyText view for the empty state.
app/src/main/java/net/ardevd/tagius/features/records/ui/list/RecordsListFragment.kt Shows/hides recycler vs. empty message based on results.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

…ecordsListFragment.kt

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@ardevd ardevd merged commit cc98b62 into main Mar 29, 2026
2 checks passed
@ardevd ardevd deleted the no-records-indicator branch March 29, 2026 21:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants