Skip to content

Fix some formatting/styling nits - #19

Merged
timwoj merged 2 commits into
masterfrom
topic/timw/format-nits
Oct 3, 2025
Merged

Fix some formatting/styling nits#19
timwoj merged 2 commits into
masterfrom
topic/timw/format-nits

Conversation

@timwoj

@timwoj timwoj commented Sep 29, 2025

Copy link
Copy Markdown
Member

This does two things

  1. Fixes the centering of the bottom section on the home page.

Before:
Screenshot 2025-09-29 at 2 55 51 PM

After:
Screenshot 2025-09-29 at 2 55 57 PM

  1. Fixes the styling of <pre> and <code> blocks to be similar to how GitHub would format them, which I feel like most people would be more familiar with. This also removes the black border around <pre> blocks. That fixes the home page to not have that weird box that looks like a search field but actually isn't.

Before:
Screenshot 2025-09-29 at 2 58 37 PM
Screenshot 2025-09-29 at 2 58 14 PM

After:
Screenshot 2025-09-29 at 2 58 44 PM
Screenshot 2025-09-29 at 2 58 18 PM

@evantypanski evantypanski left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

couple comments since when I ran it locally it didn't look right

Comment thread bropkg/webroot/css/zeek.css Outdated
Comment thread bropkg/webroot/css/zeek.css
Comment thread bropkg/webroot/css/zeek.css Outdated
Comment thread bropkg/templates/Pages/home.php
@timwoj
timwoj force-pushed the topic/timw/format-nits branch from 46dc8d6 to 7375f3c Compare September 30, 2025 20:20
@timwoj
timwoj requested a review from evantypanski September 30, 2025 21:04
@timwoj
timwoj merged commit ce7846e into master Oct 3, 2025
1 check passed
@timwoj
timwoj deleted the topic/timw/format-nits branch October 3, 2025 16:28
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