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

Update our fork of edbee-lib with latest master #7

Merged
merged 18 commits into from
Nov 9, 2020
Merged

Conversation

vadi2
Copy link
Member

@vadi2 vadi2 commented Nov 6, 2020

Unlike #6 this does not cherrypick the commits but just merges them, which is cleaner for Git history.

gamecreature and others added 18 commits July 13, 2020 18:33
Compile fix for README example
…sible incompatibility with older releases!
…r controller->setReadonly()

Update CMakeList.txt
Support for sticky-selection in replaceSelection methods. (Required for InpuMethod entry)
Improved TextEditorComponent::InputMethodEvent... It now support special chars entry like expected. (Option+e,  e => ´ => é)
- Fixed gapvector destructor: it did not use an array delete.
- TextEditorWidget::setHorizontalScrollBar not emits the correct horizontalScrollBarChanged event.
A use of the above in the test code fails to build on anything older than
Qt 5.14 as the `enum` used is only present in that and newer versions.
Previously the `enum` was a member of the `QString` class rather than the
`Qt` namespace...

Signed-off-by: Stephen Lyons <[email protected]>
…Versions

Revise: fix QString::split(...) for older Qt Versions
Copy link
Member

@SlySven SlySven left a comment

Choose a reason for hiding this comment

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

Given that our usage of Travis is fu-ed if not fubar-ed now - we should dump the CI on that platform and (ideally) replace it with something else...

@vadi2
Copy link
Member Author

vadi2 commented Nov 8, 2020

Sure. Want to approve this though?

@vadi2 vadi2 merged commit 0f8823a into Mudlet:master Nov 9, 2020
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.

4 participants