Trong các cụm từ có 2 nguyên âm, thì anh nghĩ dấu thanh nên được tự động chuyển vào nguyên âm thứ 2.

Thực ra case này vẫn hoạt động trong trường hợp "ươ" nhưng nếu là "ưo" thì nó hoạt động ko chính xác, ví dụ:

Gõ:  woj

Expected:  ưọ

Actual: ựo

cc #30
version: b627959

Given this code:

let mut out = String::new();
vi::telex::transform_buffer("wwwww".chars(), &mut out);
assert_eq!("wwww", out);

The assertion fails.

"www" is frequently typed as in "www.google.com".

Instead of re-parsing the same word over and over, we can store the word & its transformations in memory. For example:

struct Word {
  initial_consonant: String,
  vowel: String,
  final_consonant: String,
  tone_mark: ToneMark
}

Reproduce

fn main() {
    let mut result = String::new();
    for n in 1..10 {
        let mut input = Vec::new();
        for _ in 0..n {
            input.push('d');
        }
        vi::telex::transform_buffer(input.iter().cloned(), &mut result);
        result.push('\n');
    }
    println!("{}", result);
}

• Expect (similar to consecutive a's)

d
đ
dd
ddd
dddd
...

• Actual

d
đ
dd
đd
đd
đd
đd
đd
đd

Cause

• modify_letter always add tone to the first d/đ --> stuck
• if i modified so that modify_letter add tone to the last d, then reposition_letter_modification will always reorder the modifications and put the tone back to the first d/đ --> stuck
• Finally is_valid_word does not recognize dd as an invalid word (like aa). Might be best to fix here.

Input: chuyenj
Expected: chuyện
Result: chuỵên

Thiếu tính năng tự động chuyển w sau phụ âm thành ư
Ví dụ
hw -> hư

This is so that we can introduce easier definition of a typing method. Currently telex method is pretty complicated & debugging is quite difficult.

We might want to introduce new methods derived from the telex method too, so having a simple interface to define typing methods is pretty important.

Currently, when typing w after ư, we always restore it as uw.

This is true when the user first typed uw to make a ư and want to hit w again to dismiss that.

But in case the user typed w to make a ư, the next w should be restored as w not uw.

Hiện tại có 1 issues trên GõKey từ khi up lên vi-rs bản 0.6.0:

• huytd/goxkey#117

Ngoài ra còn có một vài issue khác nhưng a ko chắc có phải do vi-rs ko, có thể nó là do chức năng restore của GõKey, nhưng mà e check từ phía vi-rs thêm xem sao nhé.

• huytd/goxkey#116
• huytd/goxkey#112
• huytd/goxkey#114

After v0.3.7, we got a few panic cases, for example, when the user typing "land" in Telex mode:

thread '<unnamed>' panicked at 'Couldn't retrieve replace char for l for Dyet', vi-rs-5b567bc8b512911b/83317ad/src/processor.rs:185:36
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Ideally transform_buffer should not be panicked, but in case it is, we probably want to make it a function returning a Result, so the caller side can handle it.

huytd/goxkey#42

Hi, I use vi-rs for my linux input engine with a IBUS framework.
But IBus passes a single char each time a key typed.
For every key typed, I should display the preview string (as in Vietnameses) to users.
So I couldn't cache each/a word (Vec<char>) and pass it to transform_buffer.
So transform_buffer is called for every key typed.

So I wonder if we could create a new incremental transform_buffer.

fn transform_buffer_incremental<'def, 'buf>(
    def: &'def Definition,
    output: &'buf mut String,
) -> IncrementalBuffer;

struct IncrementalBuffer<'def, 'buf> {
    def: &'def Definition,
    output: &'buf mut String,
    input: Vec<char>,
    // cache something for example a Word.
}

impl IncrementalBuffer {
    pub fn push(&mut self, ch: char) -> TransformResult {
        self.input.push(ch);
        // ...
    }
    pub fn view(&self) -> &str;
    pub fn clear(&mut self) {
        self.output.clear();
        // clear cache
    }
}

Mô tả: Khi ở trong Insert Mode thì tự động bật chế độ gõ Tiếng Việt.
Khi quay trở lại Normal mode thì tự động tắt bộ gõ.

Currently, the replace tone mark logic works fine with "ua", for example "múa", "xúa"

But when the letter after "u" is a modified "a" like "â" "ă", the tone mark should be put on that, for example: "xúât hiện"

Note that this works fine in telex mode if I type: x u a a t s - xuất
But if I type: x u a a s t - it becomes xúât

Ví dụ bỏ dấu tự do, bỏ dấu ở cuối từ
Con cuuwf -> Con cừu
Thay vì bỏ dấu ngay sau nguyên âm
Con cuwuf -> Con cừu

Lỗi này khi mình dùng goxkey để test.
Và mình đang buồn ngủ quá nên chưa tìm ra chỗ bị bug....

hợac
hợac
hợac
hợac
ho ặc

yeah, that's it

hoặc

Mình đang tìm hiểu rust và có xem qua project Gõ Key.
Loay hoay 1 hồi thì test ra bug bị ở project này.
Hiện tại khi mình test thì nếu gõ

• Aa -> Â
• aa -> â
• AA -> AA
  Mình tạm fix bằng cách compare to_ascii_lowercase cả 2 như hình thì test thấy ok.

This code crashes unexpectedly:

    let mut out = String::new();
    let inp = "sdd";
    telex::transform_buffer(inp.chars(), &mut out);

I agree the input string is not valid.
Could transform_buffer returns an explicit error to handle this case?
So that my process -- run as daemon -- doesn't exit early.

Thank advance for your considerations.