Skip to content

Secondary language not displayed in preview#113

Merged
tarekio merged 4 commits intomainfrom
BYNT-1301-secondary-language-not-displayed-in-preview
Jun 20, 2025
Merged

Secondary language not displayed in preview#113
tarekio merged 4 commits intomainfrom
BYNT-1301-secondary-language-not-displayed-in-preview

Conversation

@apodacaduron
Copy link
Contributor

@apodacaduron apodacaduron commented Jun 17, 2025

Jira Issue

  1. BYNT-1301

Description

EventCard.js is only rendering english text in the previews, it should also render arabic translations

English Text Arabic Translation Preview
image image image

Checklist

  • Tests added/updated
  • Documentation updated (if needed)
  • New strings prepared for translations

API Changes (if applicable)

  • Permissions checked
  • Endpoint tests added

Additional Notes

[Any other relevant information]

@apodacaduron apodacaduron changed the title Bynt 1301 secondary language not displayed in preview Secondary language not displayed in preview Jun 17, 2025
@apodacaduron apodacaduron requested a review from tarekio June 17, 2025 13:46
@apodacaduron apodacaduron self-assigned this Jun 17, 2025
@tarekio tarekio requested a review from level09 June 18, 2025 14:19
Copy link
Collaborator

@level09 level09 left a comment

Choose a reason for hiding this comment

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

Looks good, I just recommend using core vuetify styles to avoid adding redundant styles

.z-1 {
z-index: 1;
}
.pi-0 {
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think we already have pe-0 in vuetify core

https://vuetifyjs.com/en/styles/spacing/#usage

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Right i dont even needed to add pi-0 even using px-0 works, my mistake, since this is already merged i'll probably change this in another PR since it's a small change

@tarekio tarekio merged commit af30802 into main Jun 20, 2025
6 of 7 checks passed
@tarekio tarekio deleted the BYNT-1301-secondary-language-not-displayed-in-preview branch June 20, 2025 14:32
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