Skip to content

Conversation

@Iximiel
Copy link
Member

@Iximiel Iximiel commented May 19, 2025

Hi @gtribello, as expected it took longer that expected:

Solving the clipping was easy, the problem is that the badges after the line "Click on the labels of the actions for more information on what each action computes" did some strange invasion in the plumed file pre element.

I changed how that line is rendered (removing the table) and now it works!
Removing that table should "clean" the boxes around the badges in the manual and clean the table borders in the other sites.

If you want add some spaces between those badges, like in the table, we can change a bit the css to add it

@gtribello
Copy link
Member

@Iximiel, thanks for looking into this. Can you look at the tests and see if you can work out what is going wrong?

@Iximiel
Copy link
Member Author

Iximiel commented May 19, 2025

I'm looking at them, I'll tag you when this is working also on the tests

@gtribello
Copy link
Member

excellent. thank you! :-)

@Iximiel
Copy link
Member Author

Iximiel commented May 19, 2025

Apparently part of this is my fault: I added a few new kw to the benchmark utilities, and that need to be updated also here
I think the latest changes make slightly easier to understand if something like this happen again. (I'll push them tomorrow)

But this only addresses half of the problems...

@Iximiel
Copy link
Member Author

Iximiel commented May 20, 2025

@gtribello now it should pass the CI

@carlocamilloni carlocamilloni merged commit 8946116 into plumed:main May 20, 2025
7 checks passed
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