Skip to content

Conversation

@shimsangjoon
Copy link
Contributor

fixing styling issue with friends list and opponents by setting the text translations as alt attributes instead of text elements

…ext translations as alt attributes instead of text elements
@shimsangjoon
Copy link
Contributor Author

Screenshot 2024-06-19 at 12 32 25

@M-DinhHoangViet
Copy link
Contributor

M-DinhHoangViet commented Jun 20, 2024

@shimsangjoon

Screenshot 2024-06-19 at 12 32 25

I think we should turn it into a button
image

@ornicar
Copy link
Collaborator

ornicar commented Jun 20, 2024

broken by #15260

…ng the text translations as alt attributes instead of text elements"

This reverts commit 2c2b52b.
@shimsangjoon
Copy link
Contributor Author

I updated the PR so the translated text is still added if shown on paginated results.
@M-DinhHoangViet not sure I understand what it is you are requesting?

Screenshot 2024-06-20 at 11 54 08 Screenshot 2024-06-20 at 11 53 55

else
api.follow(me, user.id).recoverDefault >> negotiate(
renderActions(user.name, getBool("mini")),
renderActions(user.name, getBool("mini"), getBool("showText")),
Copy link
Collaborator

Choose a reason for hiding this comment

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

?? Why is showText read from the HTTP request query parameters?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

In my initial fix I had not realised that when toggling e.g. follow/unfollow the text captions would re-appear as logic at runtime would not have the info if presented for paginated results or single user. Looking at the approach done for mini layout of the relation UI, I copied that approach.

Perhaps I could also attempt fixing this via introducing some additional routes?
e.g (/api/rel/follow/${user}/partial)

@ornicar ornicar closed this in 292d343 Jul 16, 2024
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