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

Update german translations #2

Merged
merged 3 commits into from
Mar 3, 2024
Merged

Update german translations #2

merged 3 commits into from
Mar 3, 2024

Conversation

cdfzo
Copy link

@cdfzo cdfzo commented Mar 3, 2024

hey, unfortunately it's not possible to add comments to unchanged lines (see https://github.com/orgs/community/discussions/4452), so i'm forking it instead.

  • added new translations for muted words
  • changed formal to informal
  • changed 'Du' to 'du' for consistency

could you merge this and add it to the bsky pr?
thank you

Copy link
Owner

@surfdude29 surfdude29 left a comment

Choose a reason for hiding this comment

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

Thank you so much @cdfzo, these look like great changes that will bring the German translation right up-to-date! 🙌

I just have a few small things I wanted to check before merging

src/locale/locales/de/messages.po Outdated Show resolved Hide resolved
src/locale/locales/de/messages.po Outdated Show resolved Hide resolved
src/locale/locales/de/messages.po Show resolved Hide resolved
src/locale/locales/de/messages.po Outdated Show resolved Hide resolved
src/locale/locales/de/messages.po Outdated Show resolved Hide resolved
src/locale/locales/de/messages.po Outdated Show resolved Hide resolved
@cdfzo
Copy link
Author

cdfzo commented Mar 3, 2024

@surfdude29 thank you for the review 👍

@cdfzo
Copy link
Author

cdfzo commented Mar 3, 2024

should be fixed now

@cdfzo cdfzo requested a review from surfdude29 March 3, 2024 13:35
Copy link
Owner

@surfdude29 surfdude29 left a comment

Choose a reason for hiding this comment

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

Excellent, thank you so much! 🙏

@surfdude29 surfdude29 merged commit 9e5e403 into surfdude29:update-german Mar 3, 2024
4 checks passed
@cdfzo cdfzo deleted the update-german branch March 3, 2024 14:00
surfdude29 pushed a commit that referenced this pull request Aug 9, 2024
* attempt some sort of "usurping" system

* polling-based active video approach

* split into inner component again

* click to steal active video

* disable findAndActivateVideo on native

* new intersectionobserver approach - wip

* fix types

* disable perf optimisation to allow overflow

* make active player indicator subtler, clean up video utils

* partially fix double-playing

* start working on controls

* fullscreen API

* get buttons working somewhat

* rm source from where it shouldn't be

* use video elem as source of truth

* fix keyboard nav + mute state

* new icons, add fullscreen + time + fix play

* unmount when far offscreen + round 2dp

* listen globally to clicks rather than blur event

* move controls to new file

* reduce quality when not active

* add hover state to buttons

* stop propagation of videoplayer click

* move around autoplay effects

* increase background contrast

* add subtitles button

* add stopPropagation to root of video player

* clean up VideoWebControls

* fix chrome

* change quality based on focused state

* use autoLevelCapping instead of nextLevel

* get subtitle track from stream

* always use hlsjs

* rework hls into a ref

* render player earlier, allowing preload

* add error boundary

* clean up component structure and organisation

* rework fullscreen API

* disable fullscreen on iPhone

* don't play when ready on pause

* debounce buffering

* simplify giant list of event listeners

* update pref

* reduce prop drilling

* minimise rerenders in `ActiveViewContext`

* restore prop drilling

---------

Co-authored-by: Samuel Newman <[email protected]>
Co-authored-by: Hailey <[email protected]>
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.

2 participants