Avatars borders in the Discussion MFE are square in Safari

The issue

The bug was found in safari with avatars borders, which are square. It happens because instead of borders they are using outline css property, and Safari doesn’t understand bordered outline.

Our proposal - to use border instead of outline or move outline property to pseudo element of the image parent div.

Unfortunately I’m new member and I can’t attach pictures for clearest explanation of this bug, sorry.

@arbrandes “Hi! Maybe you can help me with the problem described above? I’ve uploaded pictures to an external resource. Could you please take a look?”

Safari - https://ananasov.net/rg/discussion/image.png
Chrome - https://ananasov.net/rg/discussion/image1.png

Hey @Stanislav_Lunyachek! Welcome to the forum!

Do you mind creating a new issue on the frontend-app-discussions Github repository describing the problem? This will make it easier to find somebody to work on it, and also easier for you to keep track of it.

hi @arbrandes !! It looks like the issue in the github repo was not created, but still a good first issue for someone to fix right?

It definitely is! :slight_smile:

great, @Stanislav_Lunyachek can you please describe this a little more? What lms version is being used? also what version is used by the MFE?

I noticed that the discussions module looks pretty different in the latest version, or is there a way to configure it so it looks like you mentioned?

Hi everybody,

I have a question regarding this change that has been requested. I would like to know if this change is still relevant or necessary in the current context.

Upon reviewing the modifications, I noticed that the openedx/devstack (support) will be deprecated in the Olive releases (also it will be removed in upcoming Palm releases - Issue). As a replacement for Devstack, the recommended solution is Tutor.

Hello everyone! When I initially created this issue, we had discovered a bug in the Olive release. However, I have now been testing it on the Palm release using the open-release/palm.master tag in the Discussion MFE, and this problem is still relevant.
What should I do? Create an issue on GitHub?