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

Xóa bớt icon unikey để tránh vi phạm bản quyền #253

Merged
merged 4 commits into from
Jul 21, 2021

Conversation

binhtran432k
Copy link
Contributor

Xin lỗi AD vị sự bất tiện này, bữa mình gửi pull request nó có cả icon của Unikey trên Windown. Mặc dù là icon đó do mình vẽ nhưng dù sao thì cũng khá giống với icon trên Win. Đồng thời mình cũng optimize lại dung lượng các icon, tăng kích thước của icon bamboo mặc định, làm icon hình lá cờ trông dễ thấy hơn trên panel.

Ngoài ra, theo mình thấy thì trên linux thì svg render đẹp hơn là png khi ở trên panel. Bạn có thể xem xét lại chuyện này.
image

@luongthanhlam
Copy link
Contributor

Hi @binhtran432k, có thể nhờ bạn sửa lại icon mặc định theo định dạng svg không?

@binhtran432k
Copy link
Contributor Author

Hi @binhtran432k, có thể nhờ bạn sửa lại icon mặc định theo định dạng svg không?

@fx500ms, Khó sửa lại từ nguyên bản lắm bạn. Vẽ lại sẽ dễ hơn và mình đã vẽ lại và đặt nó nằm trong thư mục icon. Bạn có thể so sánh giữa 2 chế độ "Bamboo"(của bản gốc dưới dạng png) và "Bamboo Vietnamese"(của mình dưới dạng svg). Việc thấy svg tốt hơn chỉ mới dựa trên distro của mình. Nếu bạn so sánh thấy như vậy thật thì chỉ cần sửa lại đường dẫn icon trong bamboo.xml là xong rồi.

@luongthanhlam
Copy link
Contributor

Cool, bạn xóa icon png rồi sửa lại đường dẫn để mình merge luôn nhé 😄

@luongthanhlam
Copy link
Contributor

@binhtran432k Bamboo US layout có thể thay màu nền khác cho dễ phân biệt không nhỉ?

@binhtran432k
Copy link
Contributor Author

@binhtran432k Bamboo US layout có thể thay màu nền khác cho dễ phân biệt không nhỉ?

Mình nghĩ chắc không cần đâu, 2 cái nó chỉ khác nhau cái layout mà người dùng thì lại ít ai dùng 2 cái 1 lúc cả.

Ngoài ra mình mới xóa png và rename lại các bộ gõ để tránh trùng lặp. Mọi người vẫn chưa ai biết về vụ icons này vì chưa có thông báo lẫn wiki hưỡng dẫn nên không cần phải lo nhưng nếu bạn cài lại nhớ tháo mấy bộ gõ có icon của mình ra trước khi ibus restart nha.

@luongthanhlam luongthanhlam merged commit cd0aa9f into BambooEngine:master Jul 21, 2021
@khoaduccao
Copy link

Hi @binhtran432k, sorry mình không thấy pull request cho lần merge icon trước của bạn nên mình đành comment ở đây.
Bạn có thể add thêm icon phiên bản trắng + đen cho ibus-bamboo có được không (cho cả eng và vie)? Vì mình và một số bạn mình đang dùng theme cũng khá minimalistic, nên mong muốn có thêm icon âm bản trông cho phù hợp hơn với tổng thể giao diện.
Bạn có thể làm icon trắng đen cho bất kì icon nào của bạn hoặc icon mới cũng được ;) Thanks bạn nhiều ^^

@binhtran432k
Copy link
Contributor Author

@khoaduccao, yều của bạn mình không thể thêm vào được như mấy cái icon kia, do mỗi theme lại có 1 màu khác nhau, nếu như chỉnh trắng hoặc đen thì cũng sẽ ko hợp màu, do màu trên panel thường là xám trắng hoặc xám đen thôi. Do đó, mình mới tạo repository mới để chỉnh icon riêng màu theo ý bạn áp dụng thẳng vào icon mặc định, bạn có thể tham khảo: https://github.com/binhtran432k/ibus-bamboo-icons

@khoaduccao
Copy link

@binhtran432k Cảm ơn bạn, mình đã chỉnh và đã thành công như hình dưới :D
Script này liệu có phải chạy mỗi lần ibus-bamboo update không bạn?
Screenshot_20210830_185340

@binhtran432k
Copy link
Contributor Author

@khoaduccao, mỗi lần update nó sẽ đổi icon về mặc định, nên bạn phải chạy mỗi lần update nha. Điều này hơi phiền 1 chút, nên bạn thông cảm.

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