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

Add line height/width scale (#849) #1929

Merged
merged 8 commits into from
Oct 17, 2021
Merged

Add line height/width scale (#849) #1929

merged 8 commits into from
Oct 17, 2021

Conversation

mlouielu
Copy link
Collaborator

@mlouielu mlouielu commented Oct 12, 2021

Add line height/width scale from Vte 0.52 (#849)

Setting path: prefs/apperance/(line height scale, line width scale)

Many thanks to @egmontkob and GNOME to adding this feature in Vte.
gnunn1/tilix#1641 (comment)

Copy link
Collaborator

@Davidy22 Davidy22 left a comment

Choose a reason for hiding this comment

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

Works as expected, just needs a release notes file before approval. Also, there's some strange Gtk errors that happen when using sudo make install but not sudo make reinstall, I think entirely due to the glade version update. Works fine when running reinstall so ultimately not too much of an issue, but potentially needs to be noted for package maintainers in the release note file.

Also VTE 0.52's 3 years old now. Maybe some people still don't have it yet, maybe wait another year, but I do kind of want to drop the >= 0.52 note in the preferences window sometime because surely everyone who'll be updating to the newest guake will also have a newer version of VTE.

Copy link
Collaborator

@Davidy22 Davidy22 left a comment

Choose a reason for hiding this comment

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

Had a little time so I wrote the release note file. Merging

@Davidy22 Davidy22 merged commit cbace11 into master Oct 17, 2021
@Davidy22 Davidy22 deleted the add-line-height branch May 12, 2023 02:55
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.

2 participants