fix: show "vi" instead of lotus emoji on GNOME top bar indicator#135
Conversation
On Ubuntu 24.04 Wayland, the GNOME Shell top bar cannot render the 🪷 emoji (U+1FAB7, Unicode 14.0) used as the Vietnamese mode label, causing it to display as "?" while English correctly shows "en". Replace the emoji with "vi" to follow fcitx5 convention (en, ja, ko) and ensure compatibility across all desktop environments.
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request addresses a display compatibility issue where the Vietnamese input mode indicator, previously an emoji, failed to render correctly on certain GNOME environments. By switching to a text-based label, the change ensures that users consistently see the correct language indicator, improving the user experience and adhering to established conventions for input method display. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request changes the Vietnamese input mode label from a lotus emoji ('🪷') to the string 'vi'. This is intended to fix a rendering issue on GNOME Wayland where the emoji was not displayed correctly. The change is a one-line modification in src/lotus-engine.cpp. The new label 'vi' is consistent with conventions for other language indicators and should resolve the compatibility problem. My review of the code change did not identify any issues.
|
cái này nên để là vi nếu đang dùng gnome thôi, không nên luôn đặt là vi, bởi vì ở các DE/WM Khác thì nó hiện icon trên taskbar chứ không phải chữ |
Detect GNOME via XDG_CURRENT_DESKTOP so the indicator shows 🪷 on desktops that render emoji correctly and falls back to "vi" on GNOME. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Updated to only fall back to Changes:
|
|
@ductrantrong bạn sửa lại pr nhắm vào dev nhé, 0 được nhắm vào main. Cái này mình đã để cập trong CONTRIBUTING.md rồi |
Summary
Changes
src/lotus-engine.cpp: ChangesubModeLabelImpldefault return from "🪷" to "vi"