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

feat(doom): Add vertical center #476

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

Conversation

polirritmico
Copy link
Contributor

@polirritmico polirritmico commented Aug 7, 2024

Is working for me without any problem in a lot of config scenarios, but I think it needs more testing.

Since this use the same load_packages code from hyper, but with a return, maybe we could move that into utils or something like theme/packages?

Regards

@glepnir
Copy link
Member

glepnir commented Aug 8, 2024

how it looks like ?

@polirritmico
Copy link
Contributor Author

polirritmico commented Aug 8, 2024

how it looks like ?

Some screenshots:

  • Without config:

Screenshot_20240808_033826

  • Regular font size:

Screenshot_20240808_034052

  • Really small font:

Screenshot_20240808_034121

  • Large font:

Screenshot_20240808_034141

  • With the week day header:

Screenshot_20240808_034821

Screenshot_20240808_034911

Any particular test case?


Edit: Added context description

@polirritmico polirritmico force-pushed the doom-vertical-center branch from 54dc5da to 38aa6bb Compare August 9, 2024 20:53
@polirritmico
Copy link
Contributor Author

I've been using this code from my fork daily for two weeks without any issues.

The only question I have is whether the vertical_center option should be enabled by default or not. Since this doesn't center the preview, perhaps an opt-in approach would be better. What do you think @glepnir?

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.

Theme doom's content not vertically centered
2 participants