Skip to content

Conversation

@carns
Copy link

@carns carns commented Jun 12, 2025

Trying to see if a custom css can be used to help with #1044

@carns carns requested a review from wkliao June 12, 2025 21:02
@carns
Copy link
Author

carns commented Jun 12, 2025

I'll squash this commit so it doesn't have the history of me trying to figure out which css options to use, but @wkliao can you check the rtd build and see what you think about how this looks? This last iteration simply makes the whole document wider, which I think makes sense for the stuff we have in it. It still uses a scroll bar if the browser is not wide enough to display it entirely.

@wkliao
Copy link
Collaborator

wkliao commented Jun 12, 2025

I think we can use the css approach.
My current implementation adds an empty line to achieve the effect of line break.
It is ugly. See codes below

- Enables Darshan's non-MPI mode, required for applications that do not
call MPI_Init and MPI_Finalize.

I think using css should do this better. I will make use of this PR to do more testings.

@carns
Copy link
Author

carns commented Jun 12, 2025

Ok! If you find a better solution we can close this PR and merge something cleaner.

Along the way, I think we can remove the old *.txt and *.css files from the doc directories now that we have switched over?

@wkliao
Copy link
Collaborator

wkliao commented Jun 12, 2025

After a few tries, I think the most critical setting is the full screen width.
Please have a look.
Try adjusting your browser window size to see how the table adopts.

Copy link
Author

@carns carns left a comment

Choose a reason for hiding this comment

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

This looks great to me. It avoids wasting all of that whitespace when you are viewing the page on a computer screen, and it still adds horizontal scroll bars to the table if your browser is not wide enough.

@carns
Copy link
Author

carns commented Jun 13, 2025

Do you want me to squash and merge, or do you want to do it @wkliao ?

@wkliao
Copy link
Collaborator

wkliao commented Jun 13, 2025

Please squash and merge. Thanks.

@carns carns force-pushed the carns/dev-rtd-css-experiment branch from b190d91 to 4150156 Compare June 13, 2025 15:11
@carns carns merged commit 113f2dd into main Jun 13, 2025
6 checks passed
@carns carns deleted the carns/dev-rtd-css-experiment branch June 13, 2025 15:15
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.

3 participants