Skip to content

Flushed indentation for wrapped text on bullet point lists#124

Open
MysterieCatDev wants to merge 6 commits into
elly-code:mainfrom
MysterieCatDev:list-wrapped-indentation
Open

Flushed indentation for wrapped text on bullet point lists#124
MysterieCatDev wants to merge 6 commits into
elly-code:mainfrom
MysterieCatDev:list-wrapped-indentation

Conversation

@MysterieCatDev
Copy link
Copy Markdown
Contributor

What this does

This PR:

  • Adds flushed indentation to bullet point items that have overflowing text that wraps.
  • Adds Fedora dependency installation instructions
  • Removes a small print statement that was personally irritated me when testing :D

Fixes #120

Screenshots

image

@teamcons
Copy link
Copy Markdown
Collaborator

teamcons commented May 9, 2026

Hi!
Your screenshot has a very... Broken appearance. Is there a reason for that? I assume meson install?

Could you please open a separate PR against main for the Fedora instructions ? Ill merge immediately

I had in mind an approach but we had no time to try it out. Will try yours beforehand and see how it goes. Thank you for the time spent and efforts <3 will come back to you!

@MysterieCatDev
Copy link
Copy Markdown
Contributor Author

MysterieCatDev commented May 9, 2026

Hi! Your screenshot has a very... Broken appearance. Is there a reason for that? I assume meson install?

I wasn't sure what the issue was but I figured it was just dev-jank. I was just running the program straight from the builddev src folder. In the end I figured it was unrelated to the functionality improvements so it probably wasn't a big deal.

Could you please open a separate PR against main for the Fedora instructions ? Ill merge immediately

👍 in #125

I had in mind an approach but we had no time to try it out. Will try yours beforehand and see how it goes. Thank you for the time spent and efforts <3 will come back to you!

@teamcons
Copy link
Copy Markdown
Collaborator

Daily driving a dev build for a couple days, but this seems to hit the same issue ive met with the proof of concept in #121
As in, different zooms break the clean indentation.

Yours is a bit smarter, initial size always works, but also breaks down faster.

I had in mind to hot-fix the TextTag on zoom change, and introduce a to_indent_spacing() method for each zoom level. However did too much dev things lately so will have to wait a bit

@MysterieCatDev
Copy link
Copy Markdown
Contributor Author

Daily driving a dev build for a couple days, but this seems to hit the same issue ive met with the proof of concept in #121 As in, different zooms break the clean indentation.

Yours is a bit smarter, initial size always works, but also breaks down faster.

I had in mind to hot-fix the TextTag on zoom change, and introduce a to_indent_spacing() method for each zoom level. However did too much dev things lately so will have to wait a bit

Ahh yeah, I didn't even test with the zoom settings. Try this update and let me know what you think, basically just attempting to remeasure any time the zoom level changes.

@teamcons
Copy link
Copy Markdown
Collaborator

Looks pretty good! Running a dev build for a couple days, will check where it breaks or not

@teamcons
Copy link
Copy Markdown
Collaborator

Okay, im back

Theres a couple issues:
-Indentation isnt persistent. It is reset when app starts up
-It breaks quite horribly if user changes list prefix within the same session
-Using as list prefix the current default: " • " (two spaces, one before one after), ive noticed sometimes indenting can break spectacularly (not sure how to reproduce)

it isnt quite there but it is approaching!

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.

Bullet lists wrap / don't respect the bullet

2 participants