Skip to content
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.

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

preliminary Diff implementation for #230 (draft) #687

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Fat-Zer
Copy link

@Fat-Zer Fat-Zer commented Feb 25, 2021

TODO:

  • wxStaticText resizing (and text rewrapping) should be reimplemented.
  • Add settings for colour of diff text background in UI (people with dark UI theme will likely want to change those).
  • Add context menu option to copy old source text.

Note: It assumes dtl is installed system-wide during build. Feel free to change it if you are going to package it with tarballs.

@vslavik
Copy link
Owner

vslavik commented Apr 4, 2021

OK, I think I give up on trying to make static text work; as referred to in the other issue, the only path forward seems to be to use read-only wxTextCtrl as you suggested initially.

@cooljeanius

This comment was marked as off-topic.

@cooljeanius

This comment was marked as off-topic.

@Fat-Zer
Copy link
Author

Fat-Zer commented Jan 22, 2025

rebase upon current master and bundle dtl into tarball (there is little reason in keeping source-only library separated)

The library is header-only and rarely available system-wide, so there is
little reason not to build against the bundled version.
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.

3 participants