Skip to content

Move Labels above note text and make less visible#807

Merged
Crustack merged 2 commits into
mainfrom
feat/779-780
Jan 27, 2026
Merged

Move Labels above note text and make less visible#807
Crustack merged 2 commits into
mainfrom
feat/779-780

Conversation

@Crustack
Copy link
Copy Markdown
Owner

@Crustack Crustack commented Jan 27, 2026

Closes #779
Closes #780

Summary by CodeRabbit

  • New Features

    • Labels component added to the edit screen for better organization and management.
  • Style

    • Applied transparency styling to date elements across the app for improved visual hierarchy.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jan 27, 2026

📝 Walkthrough

Walkthrough

This pull request reorganizes the note edit screen layout by relocating the labels ChipGroup above the file preview and applying reduced opacity to secondary UI elements (date/time display) to de-emphasize non-content information and reduce visual distraction while editing.

Changes

Cohort / File(s) Summary
UI Layout Reorganization
app/src/main/res/layout/activity_edit.xml
Moved ChipGroup with id LabelGroup from Audio section to above FilesPreview RecyclerView; updated Date TextView with alpha transparency attribute
Secondary Text Styling
app/src/main/res/layout/recycler_base_note.xml
Applied alpha transparency attribute to Date TextView to reduce visibility
Resource Definitions
app/src/main/res/values/dimens.xml
Added new fraction resource alpha_secondary with value 0.65 for consistent transparency styling

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 With whiskers twitching in delight,
We moved those labels out of sight—
No longer stealing focus bright,
The date now fades, all greyed and light,
Content claims center stage tonight! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main changes: moving labels above note text and making them less visible, which directly aligns with the primary objectives.
Linked Issues check ✅ Passed All coding requirements from issues #779 and #780 are addressed: labels moved to a new ChipGroup location, alpha attribute applied to Date TextView in both layouts, and the alpha_secondary fraction resource created for de-emphasis.
Out of Scope Changes check ✅ Passed All changes are directly related to the linked issues: relocating the LabelGroup, applying transparency to the Date TextView, and adding the alpha resource for visual de-emphasis.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Crustack Crustack merged commit 4d7199b into main Jan 27, 2026
1 check passed
@Crustack Crustack deleted the feat/779-780 branch January 27, 2026 18:18
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.

Making time of modification less visible on edit screen Moving label name (pill) higher and making it less visible on edit note screen

1 participant