- Notifications
You must be signed in to change notification settings - Fork 1.1k
Fix: Refactor the CSS #17016
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
Fix: Refactor the CSS #17016
Conversation
dottydoc.css
dottydoc.css
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @Dedelweiss thanks for the PR. I don't know a ton about what CSS files should or shouldn't be included. I was told that @Florian3k is the goto Scaladoc guru at the moment, so let's wait for some input from him.
There are quite a few styles in this file, I'm not sure if we want to include them all. I will look into it. |
dottydoc.css
dottydoc.css
Hello, So, I just made some changes to the dottydoc.css file to correct some issues related to this PR. |
Unfortunately I don't know the answer to that 😅 Regarding styles in |
Indeed, I think it would be interesting to start from a base on this file and delete what is useless before getting lost and having lots of files that confront each other. |
dottydoc.css
Ho okay, I understand the difference between these two folders. The first one dotty_res/styles/themes/layout contains the CSS called in any scaladoc generated site. The docs/_assets/css folder contains custom CSS files for each project. So the CSS of dottydoc.css of the Dotty project, will not be present in the scaladoc documentations of other users. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, looks good overall 👍 I left a few minor comments.
Add import CSS Correct import
- Anchors: Remove the color of the anchors, the file content css is enough - Warning: Remove the color, banners.css is enough - Images: Add a width 100% and height auto to not overlaps
13b2c80
to a1cb207
Compare - I removed the obsolete CSS in dottydoc.css - The compliant CSS codes have been moved to folder dotty_res/styles/theme/layout - I improved the UI for the author of a blog post
Thank you very much for your feedback @Florian3k. |
I noticed that this CSS file was not imported and that this prevented the image of a blog author from having a max size for example.
Some CSS lines are not necessary and others may need to be added.
Before:
After:
As you can see, the title anchors are visible, the warning colour has disappeared or the images are still not corrected. So some modifications are necessary on this CSS file.
That's why I deleted the content of the custom file
dottydoc.css
to put it in the files of the folderdotty_res/styles/themes/layout
which are generalized to all generated documentations. This is to start from a cleaner base and add todottydoc.css
as we go along.Also, I found interesting to improve the design of Dotty's blog which is very little used (last post in 2021). For that, I modified the layout by putting the author's image at the top, in order to gather the information in a same place and make it more modern. If a separate PR is needed, I can change that.
After the modifications in the CSS file:
Fixes: #16991