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 width to guifont settings #1227

Merged
merged 5 commits into from Aug 20, 2023
Merged

Conversation

fritsstegmann
Copy link
Contributor

What kind of change does this PR introduce?

  • Feature

Allows for letter spacing by setting the width in guifont

Did this PR introduce a breaking change?

No

@MultisampledNight
Copy link
Contributor

Huh, why did you close the PR? The change could be useful for sure!

@fritsstegmann fritsstegmann reopened this Mar 4, 2022
@fritsstegmann
Copy link
Contributor Author

Ah, sorry, though you did not like it.

@fritsstegmann fritsstegmann marked this pull request as draft March 4, 2022 09:51
@fritsstegmann
Copy link
Contributor Author

marking it as draft because there are one or two things I'd like to do.

@MultisampledNight
Copy link
Contributor

Did anything change in that regard? Else I think this could be just merged.

@obmutescences
Copy link

any progress about this?

@fritsstegmann fritsstegmann marked this pull request as ready for review March 3, 2023 10:23
@fredizzimo fredizzimo mentioned this pull request Jul 5, 2023
@MultisampledNight
Copy link
Contributor

Rebased on top of main, intending to merge this. I do want to change the width behavior though, since I believe an offset would be more comfortable to use. Also need to add it to the docs that were added in meantime.

@MultisampledNight
Copy link
Contributor

Woah, quite a time ago. Sorry to have kept you waiting for a year and a half, but thank you still!! Choooooooooo choooooooooooooooo! 🚂

@MultisampledNight MultisampledNight merged commit 9b7d476 into neovide:main Aug 20, 2023
3 checks passed
@fredizzimo fredizzimo mentioned this pull request Sep 7, 2023
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.

None yet

3 participants