Interactive reminder chip to the base note cards

[ulibte]

Related Issue: #829

Interactive reminder chip to the base note cards

Added onReminderClick to open reminder settings from the base note.
Updated the base note to display the most relevant reminder.
Added visual differences for elapsed reminders on the base note: strikethrough text, 50% opacity.

2026-02-07.03-05-15.mp4

Summary by CodeRabbit

• New Features

  • Interactive reminder chip: tap to open reminder details or select in multi-select/action mode.
  • Reminder chip now shows the next relevant reminder time and visually updates when a reminder has elapsed.

• Refactor

  • Note header redesigned to display a dedicated reminder chip (replacing the previous icon/container) for clearer layout and interactions.

[coderabbitai]

Warning

Rate limit exceeded

@Crustack has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 20 minutes and 22 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📝 Walkthrough

Walkthrough

Adds a reminder click flow: ViewHolder exposes a clickable ReminderChip, ItemListener gains onReminderClick, NotallyFragment handles reminder clicks (launches RemindersActivity or toggles selection), and the note item layout replaces the RemindersView with a Material Chip.

Changes

Cohort / File(s)	Summary
Fragment Event Handling app/src/main/java/com/philkes/notallyx/presentation/activity/main/fragment/NotallyFragment.kt	Implements onReminderClick(position: Int) to toggle selection in action mode or start RemindersActivity with the note ID when not in action mode; adds import for RemindersActivity.
View Holder Reminder Logic app/src/main/java/com/philkes/notallyx/presentation/view/main/BaseNoteVH.kt	Adds setupReminderChip(baseNote) to compute/display the next relevant reminder (upcoming or latest future), format its text, apply visibility and styling (alpha/strike-through when elapsed), and wires ReminderChip click/long-click to the item listener; imports Paint and Date.
Interface Contract app/src/main/java/com/philkes/notallyx/presentation/view/misc/ItemListener.kt	Adds default fun onReminderClick(position: Int) {} to ItemListener as an optional hook for reminder clicks.
Layout Structure app/src/main/res/layout/recycler_base_note.xml	Replaces the previous header (LinearLayout TitleLayout and ImageView RemindersView) with a new Chip ReminderChip; repositions Title and Date constraints to use the chip as the header anchor.

Sequence Diagram

sequenceDiagram
    actor User
    participant VH as BaseNoteVH
    participant Frag as NotallyFragment
    participant Activity as RemindersActivity

    User->>VH: tap ReminderChip
    VH->>Frag: onReminderClick(position)
    Frag->>Frag: check action mode
    alt not in action mode
        Frag->>Activity: start RemindersActivity(noteId)
        Activity->>User: display reminders for note
    else in action mode
        Frag->>Frag: toggle/select item
    end

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 I nudged a tiny chip with my paw so spry,
Dates peep out like carrots under sky,
From holder to fragment the pathway hops,
Reminders awake — the to‑do never stops,
Hooray for little changes that make notes fly!

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 10.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'Interactive reminder chip to the base note cards' directly and clearly describes the main change: adding an interactive reminder chip component to note cards in the UI.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 2

🤖 Fix all issues with AI agents

In `@app/src/main/java/com/philkes/notallyx/presentation/view/main/BaseNoteVH.kt`:
- Around line 405-425: In setupReminderChip (BaseNoteVH), clear the
STRIKE_THRU_TEXT_FLAG when the reminder is not elapsed so recycled ViewHolders
don't keep the strike-through: replace the current paintFlags assignment inside
binding.ReminderChip with a conditional that sets paintFlags = paintFlags or
Paint.STRIKE_THRU_TEXT_FLAG when isElapsed is true, otherwise clears it with
paintFlags = paintFlags and Paint.STRIKE_THRU_TEXT_FLAG.inv(); keep the alpha
logic as-is and ensure this change is applied on binding.ReminderChip within
setupReminderChip.

In `@app/src/main/res/layout/recycler_base_note.xml`:
- Around line 76-91: Set the Chip with id ReminderChip to default visibility
GONE in XML so it doesn't flicker before bind() runs, and update its horizontal
marginStart from 10dp to 16dp to match sibling padding; locate the Chip element
with android:id="@+id/ReminderChip" in recycler_base_note.xml and make these two
attribute changes to align with how setupReminderChip toggles visibility at bind
time.

[Crustack]

Hi @ulibte, thanks so much for your contribution
Looks good to me, except I pushed 2 small changes:

• the display next notification date now takes into account if a repetition is configured
• I think it looks cleaner if the notification is displayed right below the note's title, instead of on top of it

If you are ok with these small adjustments, I'll be happy to merge this 🙂

[ulibte]

Looks great, just one thing, Android Studio is reporting this lint error: android:paddingVertical requires API level 26 (current min is 21)

here:
app/src/main/res/values/themes.xml
<item name="android:paddingVertical">0dp</item>