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

[DONE] Correct some typos + visual bug on video cards icons #1180

Merged
merged 7 commits into from
Jul 18, 2024

Conversation

Badatos
Copy link
Collaborator

@Badatos Badatos commented Jul 18, 2024

Correct smalls typos :

  • wrong quotes
  • strong instead of b
  • nbsp
  • Simplify some texts
  • 📝 Add missing help texts

  • 🐛 Correct bug on video cards icons

@Badatos Badatos self-assigned this Jul 18, 2024
Copy link
Collaborator

@AymericJak AymericJak left a comment

Choose a reason for hiding this comment

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

2 petites remarques :)

@@ -77,7 +77,9 @@ <h3>{% trans "Are you sure you want to perform a BigBlueButton live?" %}</h3>
{% endspaceless %}
{% endfor %}
</fieldset>
<small class="form-text">{% trans 'Please note: shortly after clicking the "Perform a BigBlueButton live" button, the live stream will be available to users on the <i class=\"bi bi-broadcast\"></i> <b>Lives</b> page.' %}</small><br>
<small class="form-text">
{% trans 'Please note: shortly after clicking the “Perform a BigBlueButton live” button, the live stream will be available to users on the <i class="bi bi-broadcast" aria-hidden="true"></i> <strong>Lives</strong> page.' %}
Copy link
Collaborator

Choose a reason for hiding this comment

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

></i>&nbsp;<strong>Lives</strong>

Est ce que ce serait mieux d'ajouter un &nbsp; après la balise i ?

@@ -9913,6 +9958,10 @@ msgstr "Contenu audio."
msgid "Select video “%(video_title)s”"
msgstr "Sélectionner la vidéo « %(video_title)s »"

#: pod/video/templates/videos/card_select.html
msgid "Video duration"
msgstr "Durée totale de la vidéo"
Copy link
Collaborator

Choose a reason for hiding this comment

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

C'est juste Durée de la vidéo

Copy link
Collaborator

@AymericJak AymericJak left a comment

Choose a reason for hiding this comment

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

Merci Olivier ^^

@Badatos
Copy link
Collaborator Author

Badatos commented Jul 18, 2024

Merci Olivier ^^

Merci pour tes reviews de qualité ;)

+ correct "caracters" typo
@AymericJak AymericJak merged commit 84e914f into EsupPortail:develop Jul 18, 2024
3 checks passed
@Badatos Badatos deleted the feature/correct_typos branch July 18, 2024 13:54
This pull request was closed.
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