Skip to content

Replace inline styles with Bootstrap utility classes in blog posts - #165

Merged
alamb merged 1 commit into
apache:mainfrom
kevinjqliu:kevinjqliu/use-bootstrap
Apr 1, 2026
Merged

Replace inline styles with Bootstrap utility classes in blog posts#165
alamb merged 1 commit into
apache:mainfrom
kevinjqliu:kevinjqliu/use-bootstrap

Conversation

@kevinjqliu

@kevinjqliu kevinjqliu commented Mar 28, 2026

Copy link
Copy Markdown
Contributor

Closes #157

What

Replace inline style attributes and <style> blocks with Bootstrap utility classes across 10 blog posts. The site already loads Bootstrap CSS/JS.

Changes

Inline style Bootstrap replacement
style="text-align: center" class="text-center"
width="100%" on <img> Removed (redundant with img-fluid)
<style> block for table borders Removed
<table> <table class="table table-bordered">
<td style="border:...; padding:..."> Removed (handled by table-bordered)
<blockquote style="border-left:...; padding:...; background-color:..."> class="border-start border-primary border-4 ps-3 py-2 bg-light"
style="display: flex" layouts Bootstrap grid (row/col-md-*)

Why

  • Consistent styling via the existing Bootstrap framework
  • Eliminates scattered inline styles that are harder to maintain
  • Enables future Content Security Policy (CSP) enforcement
  • Flex-to-grid migration adds responsive breakpoints for mobile

Testing

Checked before and after

localhost URL datafusion.apache.org URL
http://localhost:8000/blog/2024/01/19/datafusion-34.0.0/ http://datafusion.apache.org/blog/2024/01/19/datafusion-34.0.0/
http://localhost:8000/blog/2024/03/06/comet-donation/ http://datafusion.apache.org/blog/2024/03/06/comet-donation/
http://localhost:8000/blog/2024/08/20/python-datafusion-40.0.0/ http://datafusion.apache.org/blog/2024/08/20/python-datafusion-40.0.0/
http://localhost:8000/blog/2025/03/11/ordering-analysis/ http://datafusion.apache.org/blog/2025/03/11/ordering-analysis/
http://localhost:8000/blog/2025/03/30/datafusion-python-46.0.0/ http://datafusion.apache.org/blog/2025/03/30/datafusion-python-46.0.0/
http://localhost:8000/blog/2025/04/10/fastest-tpch-generator/ http://datafusion.apache.org/blog/2025/04/10/fastest-tpch-generator/
http://localhost:8000/blog/2025/06/30/cancellation/ http://datafusion.apache.org/blog/2025/06/30/cancellation/
http://localhost:8000/blog/2025/07/11/datafusion-47.0.0/ http://datafusion.apache.org/blog/2025/07/11/datafusion-47.0.0/
http://localhost:8000/blog/2025/07/14/user-defined-parquet-indexes/ http://datafusion.apache.org/blog/2025/07/14/user-defined-parquet-indexes/
http://localhost:8000/blog/2025/12/15/avoid-consecutive-repartitions/ http://datafusion.apache.org/blog/2025/12/15/avoid-consecutive-repartitions/

For example:
https://datafusion.apache.org/blog/2025/12/15/avoid-consecutive-repartitions/#hash-repartitioning
Screenshot 2026-03-28 at 3 44 08 PM

vs
http://localhost:8000/blog/2025/12/15/avoid-consecutive-repartitions/#hash-repartitioning
Screenshot 2026-03-28 at 3 44 23 PM

@kevinjqliu
kevinjqliu marked this pull request as draft March 28, 2026 22:41
@kevinjqliu
kevinjqliu force-pushed the kevinjqliu/use-bootstrap branch from 8e408de to 60d726e Compare March 31, 2026 18:15
@kevinjqliu
kevinjqliu marked this pull request as ready for review March 31, 2026 18:16
@kevinjqliu

Copy link
Copy Markdown
Contributor Author

@alamb this one gets rid of custom styling in favor of bootstrap util classes.

I suspect the custom styling was used because bootstrap was previously broken (fixed in #105 and #164)

@alamb alamb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me too -- I rendered this locally and manually looked at all the affected urls and they all look great

@alamb
alamb merged commit b7352a5 into apache:main Apr 1, 2026
4 checks passed
@alamb

alamb commented Apr 1, 2026

Copy link
Copy Markdown
Contributor

Thank you @kevinjqliu

@kevinjqliu
kevinjqliu deleted the kevinjqliu/use-bootstrap branch April 1, 2026 16:43
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.

Bug: Images render inconsistently on mobile (some appear too small)

2 participants