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

Markdown: Add support for webkit2gtk-4.1 #1335

Merged
merged 1 commit into from
Apr 25, 2024

Conversation

xiota
Copy link
Contributor

@xiota xiota commented Apr 24, 2024

Add support for webkit2gtk-4.1. Drop support for webkitgtk.

Still works with webkit2gtk-4.0.

Related: #1295

@elextr
Copy link
Member

elextr commented Apr 24, 2024

LGBI, the removals look good but did not check thats all that needs removing.

@elextr
Copy link
Member

elextr commented Apr 24, 2024

Needs to update README

Copy link
Member

@b4n b4n left a comment

Choose a reason for hiding this comment

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

Looks good, works fine, but the 2 first commits don't really work one without the other, so they should be merged: the first one actually make it use webkit2gtk only, but by removes the conditional that enables the code to use it (and it probably also breaks the build system by having an undefined conditional).

So fine by me once those 2 commits get squashed together.

@xiota xiota force-pushed the pr-markdown-webkit2gtk-4.1 branch from c8d5d22 to a0e8192 Compare April 24, 2024 21:51
@xiota
Copy link
Contributor Author

xiota commented Apr 25, 2024

@b4n Thanks... I've squashed the commits.

Copy link
Member

@b4n b4n left a comment

Choose a reason for hiding this comment

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

LGTM and seems to work fine here.

@b4n b4n added this to the 2.1.0 milestone Apr 25, 2024
@b4n b4n merged commit 2b897dc into geany:master Apr 25, 2024
2 checks passed
@xiota xiota deleted the pr-markdown-webkit2gtk-4.1 branch April 26, 2024 06:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants