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

[hyde theme] Re-center listings without line numbers (fixes #155 regression) #157

Conversation

hartwork
Copy link
Contributor

@hartwork hartwork commented May 21, 2018

@ralsina what do you think?

Original (before 93bae02)

hyde_original

Broken status quo

hyde_before

With pull request applied

hyde_after

@hartwork hartwork changed the title Re-center listings without line numbers (fixes #155 regression) [hyde theme] Re-center listings without line numbers (fixes #155 regression) May 21, 2018
@Kwpolska
Copy link
Member

Why not make these code blocks full-width?

@hartwork
Copy link
Contributor Author

hartwork commented May 21, 2018

It does not look cool, at least not with this very theme, it fits other themes better. If it was made full width, it would need other adjustments as well, probably a different background color, adjusted padding, maybe a single-pixel border or so. I'd rather not go there, it was beautiful until recently.

Copy link
Member

@Kwpolska Kwpolska left a comment

Choose a reason for hiding this comment

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

No, this looks terrible.

@hartwork
Copy link
Contributor Author

hartwork commented May 21, 2018

This is what listings without numbers looked like until a day ago, prior to 93bae02. This is about repair.

@Kwpolska
Copy link
Member

Okay, but let’s make it look better and make both code blocks look the same way.

@hartwork
Copy link
Contributor Author

That's a new pull request in my eyes. Let's fix the regression first, please.

@hartwork
Copy link
Contributor Author

Please please 😃

@Kwpolska
Copy link
Member

No, please do it the right way in this PR.

@hartwork
Copy link
Contributor Author

hartwork commented May 23, 2018

I disagree about what you call the right way and we have two distinct topics here. If this is a power game, I'm out.

@hartwork hartwork closed this May 23, 2018
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

2 participants