Skip to content

Conversation

@Wiechciu
Copy link
Contributor

@Wiechciu Wiechciu commented Aug 8, 2025

Pull Request Template

Title

Add Change Log into About panel

Type of Change

  • New feature

Description

This closes #34
Added a new tab into the About panel.
The text is loaded from the CHANGELOG.md file.
The text is converted from markdown to bbcode style with RegEx for headers, bold style, inline code, links and lists.

Testing

Tested that the window is showing and hiding properly and that the displayed text is showing properly with correct styling.

Impact

The text is loaded in about 0.3ms at the start of the app, so no impact on performance.

Additional Information

My solution doesn't cover all markup styling, only the ones most commonly used in the change log.
I tested the MarkdownLabel addon and it works fine as well, but it was significantly heavier on the performance (50ms vs my 0.3ms) and bring a lot more files with almost the same final effect.
If there is a need to have a full support of markdown style, this addon can be added instead and reused in the Change Log.

Checklist

  • My code adheres to the coding and style guidelines of the project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

🎉 Whoa, first pull request alert!

Huge thanks for taking the leap and contributing to Text Forge. Your involvement fuels the spirit of modularity, clarity, and collaboration that drives this project. We’ll be reviewing your work with care—expect thoughtful feedback soon! 🧠💬

Welcome aboard, and let's keep crafting the future of text editing.

@Wiechciu
Copy link
Contributor Author

Wiechciu commented Aug 8, 2025

That probably should be an Action Script instead of a standalone script, but I honestly don't quite fully understand how the Action Scripts should be implemented to make it work.

@mkh-user
Copy link
Member

mkh-user commented Aug 9, 2025

Hi @Wiechciu, welcome to Text Forge!

Your witk looks great and fast, I will make a better look very soon.

But for now, there is a mistake from me, #34 isn't about add project chagelogs in about window (I should write some information in issues...), It's about add features in editor to make writing changelogs easier for users. But changelog tab in about is a good feature, so we will merge it.

Also, please add this feature in CHANGELOG.md if is ready.

Great work!

@mkh-user mkh-user self-requested a review August 9, 2025 07:39
@mkh-user mkh-user added this to the Text Forge 0.1 milestone Aug 9, 2025
@Wiechciu
Copy link
Contributor Author

Wiechciu commented Aug 9, 2025

@mkh-user Alright, thanks!
I'll have a look and correct tomorrow.
Yeah, better descriptions of the planned features would be great, as right now there's a ton of guess work to decypher what you meant :) Looking forward to collaborate more!
Cheers!

Copy link
Contributor Author

@Wiechciu Wiechciu left a comment

Choose a reason for hiding this comment

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

Updated PR, @mkh-user please review.

@mkh-user mkh-user self-requested a review August 12, 2025 18:35
Copy link
Member

@mkh-user mkh-user left a comment

Choose a reason for hiding this comment

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

Completed as well!

@mkh-user mkh-user merged commit ca3dcf9 into text-forge:Main Aug 12, 2025
1 check passed
@mkh-user mkh-user mentioned this pull request Aug 18, 2025
6 tasks
@mkh-user mkh-user assigned mkh-user and Wiechciu and unassigned mkh-user Aug 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Completed

Development

Successfully merging this pull request may close these issues.

Show CHANGELOG in about window

2 participants