sankalp/Modus
1
0 Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases 1 Wiki Activity

refactor(src/window/notification): adjust notification markup handling and UI width constraints

Browse Source 
- Escape markup text for summary and body to prevent injection
- Simplify Label creation for body with conditional null check
- Increase max_chars_width for summary and body lines
- Remove wrap and wrap_mode settings
This commit is contained in:
sankalp committed 2026-07-06 17:34:30 +05:45
1 parent c2f9ab4070
commit 6b7348ea20
1 file changed
+26 -17
Download Patch File Download Diff File Expand all files Collapse all files
src/window/notification/notification_center.py
+26 -17
View File
@@ -118,22 +118,33 @@ class ExpandableNotificationGroup(Box):
children=[
Label(
name="notification-summary",
markup=f"<b>{self.app_name}</b>",
markup=escape_markup_text(
latest_notification._notification.summary.replace(
"\n", " "
)
),
h_align="start",
max_chars_width=25,
ellipsization="end",
),
],
),
Label(
name="notification-body",
markup=escape_markup_text(
latest_notification._notification.summary.replace(
"\n", " "
)
),
max_chars_width=25,
h_align="start",
ellipsization="end",
(
Label(
markup=escape_markup_text(
latest_notification._notification.body.replace(
"\n", " "
)
),
max_chars_width=25,
h_align="start",
ellipsization="end",
)
if latest_notification._notification.body
else Label(
markup="",
h_align="start",
)
),
],
),
@@ -543,9 +554,8 @@ class NotificationCenterWidget(NotificationWidget):
notification.summary.replace("\n", " ")
),
h_align="start",
wrap=True,
wrap_mode="word-char",
max_chars_width=30,
max_chars_width=40,
ellipsization="end",
),
],
),
@@ -555,9 +565,8 @@ class NotificationCenterWidget(NotificationWidget):
notification.body.replace("\n", " ")
),
h_align="start",
wrap=True,
wrap_mode="word-char",
max_chars_width=35,
max_chars_width=45,
ellipsization="end",
)
if notification.body
else Label(