mirror of
https://github.com/binwiederhier/ntfy.git
synced 2026-05-09 08:26:00 +02:00
[PR #804] [MERGED] Web app: add RTL support #1429
Labels
No labels
ai-generated
android-app
android-app
android-app
🪲 bug
build
build
dependencies
docs
enhancement
enhancement
🔥 HOT
in-progress 🏃
ios
prio:low
prio:low
pull-request
question
🔒 security
server
server
unified-push
web-app
website
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/ntfy#1429
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/binwiederhier/ntfy/pull/804
Author: @nimbleghost
Created: 7/3/2023
Status: ✅ Merged
Merged: 7/4/2023
Merged by: @binwiederhier
Base:
main← Head:rtl📝 Commits (2)
7a1488fWeb app: add RTL support311ffc3Format datetimes using i18n lang📊 Changes
9 files changed (+121 additions, -46 deletions)
View changed files
📝
web/package-lock.json(+36 -4)📝
web/package.json(+4 -1)📝
web/src/app/utils.js(+4 -3)📝
web/src/components/Account.jsx(+7 -8)📝
web/src/components/App.jsx(+34 -22)📝
web/src/components/Notifications.jsx(+4 -4)➕
web/src/components/RTLCacheProvider.jsx(+22 -0)📝
web/src/components/SubscriptionPopup.jsx(+8 -2)📝
web/src/components/UpgradeDialog.jsx(+2 -2)📄 Description
Resolves #663
Resolves #700
MUI has pretty good RTL support, not much to do except connect mui & i18next.
I do not read an RTL language, so probably want someone else to double check that it looks correct, but going by my basic knowledge and the Material reference, it seems fine.
https://mui.com/material-ui/guides/right-to-left
https://m2.material.io/design/usability/bidirectionality.html
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.