Closed 5andu closed 8 months ago
@5andu THANK YOU! a couple small requests before we merge:
trix.css
file, this gets included by the libraryapp/assets/stylesheets/application.tailwind.css
and app/assets/stylesheets/actiontext.css
. let's combine these into 1 file, either actiontext.css
or a new blog.css
for even more clarity. the application.tailwind.css
file should just have tailwind overrides + extensionsblog_posts/show
the <% meta title: "#{@blog_post.title} %>
(+ description) doesn't need "quoted" interpolation, just raw Ruby@ryanckulp
I'm not using the default trix.css file, I have added custom styling in the file for the editor & content, removing it breaks the styles of the editor, as well as the custom spacing I added in content. Is it ok to keep the trix.css file?
I deleted the rich text related styles from app/assets/stylesheets/application.tailwind.css
as they were unnecessary.
Fixed for the title & description, good catch thanks!
@5andu there are definitely a few custom styles, but a lot of it is duplicate from the source. we want to only host overrides and extensions, not the source. this lets us update to our Trix version without concerns.
new styling example
our trix.css
(cool)
duplication examples
our trix.css (here)
our trix.css
(notice how these look a bit different, but the variables are the same. and variable references are better than hard-coded)
next steps
we could just merge, then maybe i can help remove dupe code. my strategy would be to combine all the Trix source CSS into 1 file, then do a git diff between them, or use a web UI tool to visualize diffs.
lemme know what you think!
@ryanckulp I've deleted unnecessary classes and made the rich text more simple and functional. Here's a recap of my changes:
https://github.com/ryanckulp/speedrail/assets/38139664/7e70fa42-5508-46c0-b7ae-999e934612c7
excellent work, thanks for the second push. merging.
https://github.com/ryanckulp/speedrail/assets/38139664/eebe0ac6-90fb-4d6d-99c9-8e878ceb6a0a