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

Add warning for overlapping highlights #2215

Merged
merged 5 commits into from
May 20, 2024

Conversation

RoyEJohnson
Copy link
Contributor

@RoyEJohnson RoyEJohnson commented Apr 9, 2024

@RoyEJohnson RoyEJohnson requested a review from a team as a code owner April 9, 2024 14:12
@RoyEJohnson RoyEJohnson requested a review from Dantemss April 9, 2024 14:12
@TomWoodward TomWoodward temporarily deployed to rex-web-add-warning-for-kwnxgo April 9, 2024 14:12 Inactive
@TomWoodward TomWoodward temporarily deployed to rex-web-add-warning-for-kwnxgo April 9, 2024 19:42 Inactive
Copy link
Member

@Dantemss Dantemss left a comment

Choose a reason for hiding this comment

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

During testing make sure the warning doesn't trigger if you make the text selection empty or anything weird like that

@TomWoodward TomWoodward temporarily deployed to rex-web-add-warning-for-kwnxgo April 9, 2024 20:01 Inactive
@TomWoodward TomWoodward temporarily deployed to rex-web-add-warning-for-kwnxgo April 10, 2024 13:59 Inactive
@TomWoodward TomWoodward temporarily deployed to rex-web-add-warning-for-kwnxgo April 10, 2024 19:16 Inactive
@TomWoodward TomWoodward temporarily deployed to rex-web-add-warning-for-kwnxgo April 11, 2024 14:12 Inactive
@TomWoodward TomWoodward temporarily deployed to rex-web-add-warning-for-kwnxgo April 11, 2024 19:27 Inactive
@Malar-Natarajan Malar-Natarajan temporarily deployed to rex-web-add-warning-for-hj7p2f April 19, 2024 15:16 Inactive
@TomWoodward TomWoodward temporarily deployed to rex-web-add-warning-for-hj7p2f April 19, 2024 17:29 Inactive
@TomWoodward TomWoodward temporarily deployed to rex-web-add-warning-for-hj7p2f April 22, 2024 16:22 Inactive
@TomWoodward TomWoodward temporarily deployed to rex-web-add-warning-for-hj7p2f April 23, 2024 14:51 Inactive
@TomWoodward TomWoodward temporarily deployed to rex-web-add-warning-for-hj7p2f April 23, 2024 16:40 Inactive
@Malar-Natarajan Malar-Natarajan temporarily deployed to rex-web-add-warning-for-sjgvs5 April 30, 2024 19:15 Inactive
@TomWoodward TomWoodward temporarily deployed to rex-web-add-warning-for-sjgvs5 May 1, 2024 19:02 Inactive
@TomWoodward TomWoodward temporarily deployed to rex-web-add-warning-for-sjgvs5 May 2, 2024 19:35 Inactive
@TomWoodward TomWoodward temporarily deployed to rex-web-add-warning-for-sjgvs5 May 2, 2024 20:51 Inactive
@TomWoodward TomWoodward temporarily deployed to rex-web-add-warning-for-sjgvs5 May 2, 2024 21:31 Inactive
@TomWoodward TomWoodward temporarily deployed to rex-web-add-warning-for-sjgvs5 May 3, 2024 14:32 Inactive
@TomWoodward TomWoodward temporarily deployed to rex-web-add-warning-for-sjgvs5 May 3, 2024 14:33 Inactive
@TomWoodward TomWoodward temporarily deployed to rex-web-add-warning-for-sjgvs5 May 6, 2024 18:57 Inactive
@RoyEJohnson RoyEJohnson force-pushed the add-warning-for-overlapping-highlights branch from cf0eb83 to 4a249fd Compare May 6, 2024 20:23
@TomWoodward TomWoodward temporarily deployed to rex-web-add-warning-for-sjgvs5 May 6, 2024 20:23 Inactive
@TomWoodward TomWoodward temporarily deployed to rex-web-add-warning-for-sjgvs5 May 6, 2024 21:44 Inactive
@TomWoodward TomWoodward temporarily deployed to rex-web-add-warning-for-sjgvs5 May 7, 2024 20:34 Inactive
@RoyEJohnson RoyEJohnson force-pushed the add-warning-for-overlapping-highlights branch from e68873d to da7c608 Compare May 8, 2024 20:22
@TomWoodward TomWoodward temporarily deployed to rex-web-add-warning-for-sjgvs5 May 8, 2024 20:23 Inactive
@RoyEJohnson RoyEJohnson force-pushed the add-warning-for-overlapping-highlights branch from da7c608 to f73b598 Compare May 8, 2024 21:44
@TomWoodward TomWoodward temporarily deployed to rex-web-add-warning-for-sjgvs5 May 8, 2024 21:45 Inactive
@@ -125,6 +125,7 @@
"i18n:notification:toast:highlights:popup:load-failure": "No se pudo cargar algunos textos resaltados. Por favor, compruebe su conexión y vuelva a intentarlo.",
"i18n:notification:toast:highlights:popup:print-failure": "No podemos imprimir esta página. Por favor, compruebe su conexión y vuelva a intentarlo.",
"i18n:notification:toast:study-guides:load-failure": "No podemos cargar las guías de estudio en esta página. Por favor, compruebe su conexión y vuelva a intentarlo.",
"i18n:notification:toast:highlights:select-overlap": "No se admiten resaltados superpuestos.",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@bethshook Can you check this translation, please?

Copy link
Contributor

Choose a reason for hiding this comment

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

looks good

@@ -125,6 +125,7 @@
"i18n:notification:toast:highlights:popup:load-failure": "Niektóre zakreślenia na tej stronie nie mogą zostać załadowane. Sprawdź połączenie z internetem i spróbuj ponownie.",
"i18n:notification:toast:highlights:popup:print-failure": "Nie możemy wydrukować tej strony. Sprawdź połączenie z internetem i spróbuj ponownie.",
"i18n:notification:toast:study-guides:load-failure": "Nie możemy załadować zawartości Kompendium na tej stronie. Sprawdź połączenie z internetem i spróbuj ponownie.",
"i18n:notification:toast:highlights:select-overlap": "Nakładające się podświetlenia nie są obsługiwane.",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@paulinaszczucinska Can you check this translation of "Overlapping highlights are not supported" ?

Choose a reason for hiding this comment

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

Sure @RoyEJohnson, it would be: "Nakładające się zaznaczenia nie są obsługiwane."

@RoyEJohnson RoyEJohnson force-pushed the add-warning-for-overlapping-highlights branch from f73b598 to d35ea7e Compare May 13, 2024 21:19
@TomWoodward TomWoodward temporarily deployed to rex-web-add-warning-for-sjgvs5 May 13, 2024 21:20 Inactive
@Malar-Natarajan Malar-Natarajan temporarily deployed to rex-web-add-warning-for-sjgvs5 May 16, 2024 15:20 Inactive
@Malar-Natarajan
Copy link
Contributor

@RoyEJohnson The unit tests are failing in this PR. Could you please have a look

@Malar-Natarajan Malar-Natarajan temporarily deployed to rex-web-add-warning-for-sjgvs5 May 16, 2024 18:24 Inactive
@RoyEJohnson RoyEJohnson force-pushed the add-warning-for-overlapping-highlights branch from ccd3805 to c3e771a Compare May 17, 2024 16:37
@TomWoodward TomWoodward temporarily deployed to rex-web-add-warning-for-sjgvs5 May 17, 2024 16:37 Inactive
@Malar-Natarajan Malar-Natarajan temporarily deployed to rex-web-add-warning-for-sjgvs5 May 17, 2024 21:01 Inactive
@Malar-Natarajan Malar-Natarajan merged commit 32562dc into main May 20, 2024
8 of 9 checks passed
@Malar-Natarajan Malar-Natarajan deleted the add-warning-for-overlapping-highlights branch May 20, 2024 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants