Skip to content

Fixed-size notes support#641

Merged
vkareh merged 1 commit into
mate-desktop:masterfrom
qmaldon:stickynotes-support-fixedsize
Jun 3, 2026
Merged

Fixed-size notes support#641
vkareh merged 1 commit into
mate-desktop:masterfrom
qmaldon:stickynotes-support-fixedsize

Conversation

@qmaldon
Copy link
Copy Markdown
Contributor

@qmaldon qmaldon commented Sep 5, 2022

Hi all,

These changes apply to stickynotes only.

This is my first PR on a Mate repo. Please bear with me, since there are a few things that are not straightforward:

  1. Multiple changes were done automatically by the build system into .po files.
    I understand these are translation files caused by a change I did in stickynotes/sticky-notes-preferences.ui
  2. All changes were verified manually in my host (RHEL 8). I'm not sure if your CI/CD system will kick in during PR.
  3. Is there a way to get this branch backported to 1.26?

What is being enhanced & fixed:

  1. [Enhanced] Support for fixed size notes. That is, a mechanism to disallow notes resizing (on all notes).
  2. [Bug fix] On a large note (vertical length > screen height), after restarting the stickynotes applet there are no verifications of the w,h sizes stored in the xml file; as consequence no automatic scrollbar is set, leading to an ugly and difficult to manage note.
    This is fixed now, since during startup a check based on screen size is done (propagated from existed code).

Please look at the following images to get an idea of the code changes:

Screenshot from 2022-09-04 20-09-07

Screenshot from 2022-09-04 20-09-27

I decided to make these changes since the stickynotes applet has been one of my main productivity tools since 2014 (I used to run RHEL 5 at that time).

Please let me know if there is any change needed or edit as you see fit.

Thanks

Quintin Maldonado

@mbkma
Copy link
Copy Markdown
Member

mbkma commented Sep 5, 2022

Hi, thanks for contributing! Please do not change po files here. This will be overwritten by transifex anyway. To untrack po files, I belive it is
git rm -r --cached <folder>
I am not sure though.

@qmaldon qmaldon force-pushed the stickynotes-support-fixedsize branch from bcc5d40 to 37c0932 Compare September 5, 2022 19:12
@qmaldon
Copy link
Copy Markdown
Contributor Author

qmaldon commented Sep 5, 2022

Hi, thanks for contributing! Please do not change po files here. This will be overwritten by transifex anyway. To untrack po files, I belive it is git rm -r --cached <folder> I am not sure though.

Hi @mbkma ,
Removed .po changes from branch as suggested. Please have a look.

I think there is an issue in the debian build. Fedora looks good.

Is there a way to get this branch backported to 1.26?

Following up on this, perhaps it would be 1.26.2?

Thanks

Quintin Maldonado

Comment thread stickynotes/stickynotes.c
Comment thread stickynotes/stickynotes.c Outdated
@vkareh vkareh force-pushed the stickynotes-support-fixedsize branch 3 times, most recently from c27b125 to e6e709b Compare June 3, 2026 21:04
Add a new preference to force fixed size on all notes to prevent
resizing. Also fix a bug where notes taller than the screen height would
not display scrollbars after applet restart.

Fixes mate-desktop#370
@vkareh vkareh force-pushed the stickynotes-support-fixedsize branch from e6e709b to 3348bbf Compare June 3, 2026 21:06
@vkareh vkareh linked an issue Jun 3, 2026 that may be closed by this pull request
Copy link
Copy Markdown
Member

@vkareh vkareh left a comment

Choose a reason for hiding this comment

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

I rebased from master, fixed the merge conflicts and the indentation/spacing issues. It works well for me and fixes the notes resizing automatically when you add text to them (which is really annoying).

@vkareh vkareh merged commit 1c9a5e6 into mate-desktop:master Jun 3, 2026
5 checks passed
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.

feature request - sticky notes applet word wrap

4 participants