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

Fix: Some display issues #2219

Merged
merged 2 commits into from
Dec 18, 2023
Merged

Fix: Some display issues #2219

merged 2 commits into from
Dec 18, 2023

Conversation

Hima-Zinn
Copy link
Contributor

(English ver. by BIng Translation)
cf: #2209 (comment) ( @tkusano )

Dear Administrators of this Repository,
Hello.

When I was working on Japanese localization ( #2209 (comment) ), a colleague discovered a problem that the GUI display was distorted in the early version of Japanese and that "Profile" was not displayed in Japanese.

With this fix, "NewPost" and "Profile" will be displayed correctly in the selected language, and other languages will be served more smoothly.

I hope you will join us in Merge.
Hima

(Original (Japanese))
関連: #2209 (comment) ( @tkusano )

このリポジトリの管理者のみなさん
こんにちは。

日本語のローカライズ作業をしていた( #2209 (comment) )際、日本語の初期バージョンでGUIの表示が乱れ、また"Profile"が日本語で表示されない問題を仲間が発見してくれました。

この修正により、"NewPost"と"Profile"が選択した言語で正しく表示されるようになり、また他の言語の提供もスムーズになると思われます。

ぜひ、Mergeしていただければ幸いです。
Hima

@tkusano
Copy link
Contributor

tkusano commented Dec 16, 2023

I propose a CSS fix to correctly display the translated versions of the Feedback · Privacy · Terms · Help links in the desktop right navigation.
Without this patch, each link is word-wrapped in Japanese and looks ugly.

Before patch:
image

After this patch applied:
image

diff --git a/src/view/shell/desktop/RightNav.tsx b/src/view/shell/desktop/RightNav.tsx
index c126935d..8d9961a5 100644
--- a/src/view/shell/desktop/RightNav.tsx
+++ b/src/view/shell/desktop/RightNav.tsx
@@ -52,7 +52,7 @@ export function DesktopRightNav() {
               </Text>
             </View>
           ) : undefined}
-          <View style={[s.flexRow]}>
+          <View style={[{flexWrap: 'wrap'}, s.flexRow]}>
             {hasSession && (
               <>
                 <TextLink

@Hima-Zinn
Copy link
Contributor Author

Thank you very much.
I will work on incorporating those corrections as well, so please be patient.

@Hima-Zinn
Copy link
Contributor Author

@tkusano
I have corrected the points you have pointed out.
Please check it.

In addition, since the content handled is now general display systems, I will change the title to an appropriate one.

@Hima-Zinn Hima-Zinn changed the title Fix: Change the width of "New Post" and the string of "Profile" Fix: Some display issues Dec 16, 2023
@Hima-Zinn
Copy link
Contributor Author

Hima-Zinn commented Dec 16, 2023

@ansh
Since this issue is combined with the recent correction of the Japanese translation ( #2209 ), I would appreciate it if you could merge both of them at the same time.

@ansh
Copy link
Contributor

ansh commented Dec 16, 2023

This looks good. Thank you @Hima-Zinn

Copy link
Contributor

@tkusano tkusano left a comment

Choose a reason for hiding this comment

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

checked

Copy link
Contributor

@ansh ansh left a comment

Choose a reason for hiding this comment

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

Changes look good to me.

@ansh ansh merged commit b942da1 into bluesky-social:main Dec 18, 2023
4 checks passed
estrattonbailey added a commit that referenced this pull request Dec 18, 2023
* origin/main: (22 commits)
  basic public RSS feed for profiles (#2229)
  use `s.likeColor` everywhere (#2234)
  Add credits to localization doc (#2233)
  Fix `Logotype` fill for dark mode (#2230)
  Hide label appeal on self-labeled posts (#2232)
  Fix & Add: Japanese Translation (3) (#2226)
  tweak social card meta yet again (#2228)
  bskyweb: update golang indigo dep (from May!), and some small devex tweaks (#2227)
  Fix: Some display issues (#2219)
  Update splash screen tagline, update translations, bump ios build number
  Bump ios build number
  Tweaks (#2225)
  1.60
  Traffic reduction and tuned caching strats (#2215)
  Super secret changes don't look (#2218)
  Fix android icon dims (#2213)
  Log post creation failures (#2205)
  Some brand, some pwi (#2212)
  🤫  (#2211)
  PWI behavior updates (#2207)
  ...
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