Skip to content

fix: restore native Frappe v15 notification presentation - #18

Merged
pnwilly merged 8 commits into
mainfrom
fix/native-v15-notification-content
Sep 3, 2026
Merged

pnwilly merged 8 commits into
mainfrom
fix/native-v15-notification-content

Conversation

@pnwilly

@pnwilly pnwilly commented Sep 3, 2026

Copy link
Copy Markdown
Owner

Summary

  • restore Frappe v15's native notification message rendering
  • restore Frappe v15's vertical row padding and margins
  • retain the avatar grid, mark-as-read and per-row delete controls
  • retain additional right padding so the added controls do not overlap messages
  • bump assets and package version to 0.0.12

Verification

  • no custom content sanitiser, line clamp or media-hiding selector remains
  • row padding is 10px vertically and row margins are 10px, matching Frappe v15
  • delete and mark-as-read behaviour remains wired

@coderabbitai

coderabbitai Bot commented Sep 3, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: 087f0afe-938d-4b4f-9808-9321a1297486


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.

@pnwilly pnwilly changed the title fix: restore native Frappe v15 notification content fix: restore native Frappe v15 notification presentation Sep 3, 2026
@pnwilly
pnwilly merged commit f832bf1 into main Sep 3, 2026
5 checks passed
@pnwilly
pnwilly deleted the fix/native-v15-notification-content branch September 3, 2026 10:05
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.

1 participant