Skip to content

style(stickies): apply placeholder text color to checked checklist items#90

Merged
Rafetikus merged 6 commits into
mainfrom
89-fix-sticky-notes-fade-in-done
Apr 15, 2026
Merged

style(stickies): apply placeholder text color to checked checklist items#90
Rafetikus merged 6 commits into
mainfrom
89-fix-sticky-notes-fade-in-done

Conversation

@Rafetikus
Copy link
Copy Markdown
Member

@Rafetikus Rafetikus commented Apr 15, 2026

This pull request enhances the StickyNoteCard component by adding helpful tooltips to the formatting and action buttons, improving accessibility and user experience. It also introduces platform-aware keyboard shortcut hints and visually distinguishes completed checklist items. The most important changes are grouped below:

User Experience Improvements:

  • Added Tooltip components to formatting buttons (Bold, Italic, To-do list) and the color/background and delete buttons, providing descriptive labels and shortcut hints. [1] [2] [3] [4] [5] [6]
  • Displayed keyboard shortcuts in tooltips, dynamically showing Cmd or Ctrl based on the user's platform for better clarity.

Visual Enhancements:

  • Styled completed checklist items in sticky notes to use placeholder-colored text, matching the Plane design, and added a color transition for checklist items.
  • Added a dedicated class name (sticky-note-editor-content) to the editor content for more targeted styling.

Code Maintenance:

@Rafetikus Rafetikus linked an issue Apr 15, 2026 that may be closed by this pull request
Copilot AI review requested due to automatic review settings April 15, 2026 07:04
@Rafetikus Rafetikus self-assigned this Apr 15, 2026
@Rafetikus Rafetikus added this to the Deadline milestone Apr 15, 2026
@nazarli-shabnam nazarli-shabnam added bug Something isn't working enhancement New feature or request labels Apr 15, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates sticky note checklist styling so completed (checked) task items render using the placeholder text color with a smooth color transition, improving visual consistency with Plane.

Changes:

  • Add a dedicated sticky-note-editor-content class to the sticky note EditorContent container for targeted styling.
  • Introduce global CSS rules to style checked task list items (li[data-checked='true']) inside sticky notes using --txt-placeholder with a color transition.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
ui/src/index.css Adds sticky-note-scoped CSS selectors to apply placeholder-colored text and transitions for checked task items.
ui/src/components/stickies/StickyNoteCard.tsx Adds sticky-note-editor-content class to enable scoped styling of the editor content.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread ui/src/index.css Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread ui/src/components/stickies/StickyNoteCard.tsx Outdated
Comment thread ui/src/components/stickies/StickyNoteCard.tsx
Comment thread ui/src/components/stickies/StickyNoteCard.tsx
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread ui/src/components/stickies/StickyNoteCard.tsx
Comment thread ui/src/components/stickies/StickyNoteCard.tsx
@Rafetikus Rafetikus requested a review from Copilot April 15, 2026 08:25
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread ui/src/components/stickies/StickyNoteCard.tsx
Copy link
Copy Markdown
Member

@nazarli-shabnam nazarli-shabnam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can be merged. congrats

Copy link
Copy Markdown
Member

@Javenn0 Javenn0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice Job!

Copy link
Copy Markdown
Member

@martian56 martian56 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well done

@Rafetikus Rafetikus merged commit de3e623 into main Apr 15, 2026
11 checks passed
@martian56 martian56 deleted the 89-fix-sticky-notes-fade-in-done branch May 2, 2026 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix: Sticky Notes fade in done

5 participants