Skip to content
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] List paragraphs inline 100%, image into lists #6273

Merged
merged 3 commits into from
Mar 18, 2023

Conversation

pgi-jsanchez
Copy link
Contributor

Hello.

Editing in a private wiki, I try to use images to be part of a list, the issue is that those got aligned with the text, so I tested with the css to fix it. I also thought to change it with display:block;, but if there's a reason to use inline-block for this, this change still will respect that and fix this issue. I include this tests images.

Before:
Captura de pantalla-20230317151347-784x340

After:
Captura de pantalla-20230317151307-908x422

@NGPixel
Copy link
Member

NGPixel commented Mar 17, 2023

This makes many assumptions that are incorrect.

  • Your change applies to all element types, regardless of whether they are an image or not.
  • What if the image is smaller?

@pgi-jsanchez
Copy link
Contributor Author

Does not apply to all, only for lists first paragraph, that actually contains the text. Adding a css that changes this paragraph when next paragraph contains an image, might fail if there are nested lists or other text, so with this, can help in this situation.

I must admit that images changes the sizes a little bit if using =%x, but I tested this change and using this parameter, and works just fine.

Captura de pantalla-20230317171817-745x509

@NGPixel NGPixel merged commit 4de461f into requarks:main Mar 18, 2023
davidflypei pushed a commit to davidflypei/wiki that referenced this pull request Jun 13, 2023
* Adds width 100% for ol paragraph

---------

Co-authored-by: Nicolas Giard <github@ngpixel.com>
jionggyu pushed a commit to jionggyu/wiki-2.5.302-patch that referenced this pull request Jul 9, 2024
* Adds width 100% for ol paragraph

---------

Co-authored-by: Nicolas Giard <github@ngpixel.com>
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.

2 participants