Skip to content

🎨 Palette: Refine system notification copy #116

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
manupawickramasinghe wants to merge 2 commits into
base: main
Choose a base branch
from
Open

🎨 Palette: Refine system notification copy #116

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
e685442
Refine foreground service notification copy
google-labs-jules[bot] May 22, 2026
aefbd56
Refine foreground service notification copy
google-labs-jules[bot] May 23, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions .Jules/palette.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -146,3 +146,6 @@
## 2026-05-20 - Mouse Hover States for Custom Selectors
**Learning:** Custom drawable selectors (like `ic_shutter.xml`) on Android often define `state_pressed` and `state_focused` but omit `state_hovered`. This strips visual feedback for users navigating with pointer devices (mice, trackpads) on environments like Chromebooks or Samsung DeX, degrading the user experience compared to native components.
**Action:** Always include `android:state_hovered="true"` alongside focus and pressed states in custom interactive background selectors to ensure universal visual feedback across all input methods.
## 2024-05-22 - Tone and Voice in System Notifications
**Learning:** Hardcoding technical or aggressive terminology like "Kill" for destructive actions in user-facing system notifications (like foreground service controls) degrades the user experience by using unnecessarily alarming language instead of standard, culturally understood UX terms.
**Action:** Always avoid aggressive technical jargon in UI copy. Opt for standard UX phrasing like "Stop" instead of "Kill", and "Tap" instead of "Click" for mobile contexts, and ensure all notification text is extracted to localized string resources.
10 changes: 5 additions & 5 deletions app/src/main/java/com/samsung/android/scan3d/serv/Cam.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,10 +45,10 @@ class Cam : Service() {
"start" -> {
val channel = NotificationChannel(
CHANNEL_ID,
CHANNEL_ID,
getString(R.string.notification_channel_name),
NotificationManager.IMPORTANCE_DEFAULT
)
channel.description = "RemoteCam run"
channel.description = getString(R.string.notification_channel_desc)
val notificationManager = getSystemService(NotificationManager::class.java)
notificationManager.createNotificationChannel(channel)

Expand All @@ -72,9 +72,9 @@ class Cam : Service() {

val builder =
NotificationCompat.Builder(this, CHANNEL_ID)
.setContentTitle("RemoteCam (active)")
.setContentText("Click to open").setOngoing(true)
.setSmallIcon(R.drawable.ic_linked_camera).addAction(R.drawable.ic_close, "Kill",pendingIntentKill)
.setContentTitle(getString(R.string.notification_title))
.setContentText(getString(R.string.notification_text)).setOngoing(true)
.setSmallIcon(R.drawable.ic_linked_camera).addAction(R.drawable.ic_close, getString(R.string.notification_action_stop), pendingIntentKill)
.setContentIntent(pendingIntent)


Expand Down
7 changes: 7 additions & 0 deletions app/src/main/res/values/strings.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,13 @@
<string name="copied_to_clipboard">Copied to clipboard</string>
<string name="error_no_browser">No web browser found. Please install a browser to view the repository.</string>

<!-- Notification Strings -->
<string name="notification_channel_name">RemoteCam Service</string>
<string name="notification_channel_desc">Shows active status of the remote camera server</string>
<string name="notification_title">RemoteCam (active)</string>
<string name="notification_text">Tap to open app</string>
<string name="notification_action_stop">Stop</string>

<!-- Stop Confirmation Dialog -->
<string name="stop_confirmation_title">Stop Server?</string>
<string name="stop_confirmation_message">This will stop the camera stream and close the application.</string>
Expand Down