search

teni-ime / ibus-teni

Bộ gõ tiếng Việt cho Linux chạy trên IBus
https://www.facebook.com/teni.ime
GNU General Public License v3.0
7 stars 0 forks source link

auto commit text after 3s of inactive, resolve #45 #63

Closed luongthanhlam closed 6 years ago

openhoangnc commented 6 years ago

@luongthanhlam Về PR này mình sẽ xem xét sau.

openhoangnc commented 6 years ago

@luongthanhlam Việc bạn copy mã nguồn, sửa tên thành ibus-bamboo và không ghĩ rõ nguồn gốc source là vi phạm nghiêm trọng điều khoản mã nguồn mở.

openhoangnc commented 6 years ago

về nội dung PR

3 giây có thuyết phục hơn 1 giây hay 10 giây không?

ghinlop commented 6 years ago

Có thể cho phép user tự diều chỉnh thời gian commit được không @openhoangnc ?

openhoangnc commented 6 years ago

@ghinlop Cho người dùng điều chỉnh thời gian thì có thể làm được. Mình nghĩ rằng việc này không cần thiết nữa, người dùng chỉ cần thích nghi một chút là dùng được.

luongthanhlam commented 6 years ago

@openhoangnc mình đã cập nhật thêm bản quyền tác giả, nếu thấy chưa hợp lý phiền bạn góp ý thêm hoặc tạo pull request. Thanks!

openhoangnc commented 6 years ago

@luongthanhlam cảm ơn bạn đã cập nhật. Mọi đóng góp đều được hoan nghênh, tuy nhiên nội dung PR này mình thấy chưa cần thiết.

Hiện tại đang còn một số issue mà nhiều người gặp nhất là lỗi khi gõ nhanh và lỗi con trỏ nhảy về đầu dòng, nếu có thời gian thì bạn có thể tìm cách giải quyết và chia sẻ cho cộng đồng.

luongthanhlam commented 6 years ago

mình chưa check kĩ nhưng có vẻ lỗi con trỏ liên quan đến issue này https://github.com/facebook/draft-js/pull/1419

luongthanhlam commented 6 years ago

@openhoangnc check giúp mình PR#64 nhé