[bug] List items with inner p tags don't have bullets #348

Closed
opened 2023-10-06 13:43:28 +00:00 by xarvos · 5 comments
Contributor

Version

e7a558a

What were you trying to do?

I was viewing posts from a firefish instance with a list.

What did you expect to happen?

The list has bullets

What actually happened?

The list appears without bullets.

After a few HTML inspects, I find out that <li> elements from firefish has <p> inside, and I can reproduce that within akkoma by composing a post in HTML. This happens for both unordered and ordered lists, and does not happend on other FE such as mastodon FE/glitch-soc.

Screenshots of a test post from akkoma and glitch-soc for referrence:

list with p tag appearing without bullets and numbers on akkoma

the same posts with bullets and numbers on glitch-soc

Severity

I can manage

Have you searched for this issue?

  • I have double-checked and have not found this issue mentioned anywhere.
### Version e7a558a ### What were you trying to do? I was viewing posts from a firefish instance with a list. ### What did you expect to happen? The list has bullets ### What actually happened? The list appears without bullets. After a few HTML inspects, I find out that `<li>` elements from firefish has `<p>` inside, and I can reproduce that within akkoma by composing a post in HTML. This happens for both unordered and ordered lists, and does not happend on other FE such as mastodon FE/glitch-soc. Screenshots of [a test post](https://outerheaven.club/notice/AaUjVWgupxEKFgPdpI) from akkoma and glitch-soc for referrence: ![list with p tag appearing without bullets and numbers on akkoma](/attachments/9d94019d-c172-44db-952a-ad01d216483a) ![the same posts with bullets and numbers on glitch-soc](/attachments/30a37d54-1611-493d-b4c0-19f790c444a1) ### Severity I can manage ### Have you searched for this issue? - [ ] I have double-checked and have not found this issue mentioned anywhere.
Member

image

Looks fine to me? :thonk:

![image](/attachments/8db288e4-35bc-4254-b584-45b365027ecf) Looks fine to me? :thonk:
Author
Contributor

It looks like this is only a problem on firefox. Other clients work fine on firefox though

It looks like this is only a problem on firefox. Other clients work fine on firefox though
Author
Contributor

If I disable this rule on firefox, the bullets and numbers are also displayed fine 🤔

.StatusBody .text > * {
	white-space: pre-wrap;
}

no idea why this causes that. I also tried to reproduce this in isolation with the HTML below but it didn't work either.

<style>
ul {
	white-space: pre-wrap;
}
</style>
<ul>
	<li><p>test list</p></li>
</ul>
If I disable this rule on firefox, the bullets and numbers are also displayed fine 🤔 ```css .StatusBody .text > * { white-space: pre-wrap; } ``` no idea why this causes that. I also tried to reproduce this in isolation with the HTML below but it didn't work either. ```html <style> ul { white-space: pre-wrap; } </style> <ul> <li><p>test list</p></li> </ul> ```
Member

Firefox ESR 115 is still affected, but Firefox 123 and mobile Firefox display the bullet point and number.
Guess this was a bug with Firefox’s default styling

Firefox ESR 115 is still affected, but Firefox 123 and mobile Firefox display the bullet point and number. Guess this was a bug with Firefox’s default styling
Author
Contributor

classic rage comic: it doesn't work... why? it works... why?

just checked that firefox 122 also works.

Guess this was a bug with Firefox’s default styling

yea i suppose that it's a bug on firefox's side that's fixed after all, but what perplexed me was that i couldn't reproduce that outside akkoma

![classic rage comic: it doesn't work... why? it works... why?](https://d.justpo.st/media/images/2013/03/463e4e4f4acb9afc185a067b7f360321.jpg) just checked that firefox 122 also works. > Guess this was a bug with Firefox’s default styling yea i suppose that it's a bug on firefox's side that's fixed after all, but what perplexed me was that i couldn't reproduce that outside akkoma
Sign in to join this conversation.
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: AkkomaGang/akkoma-fe#348
No description provided.