Andre Poenitz wrote:
The LFUNs are named wrongly. It should be 'LFUN_CHAR_LEFT', since that's what's supposed to happen on screen. This could be mapped to LFUN_CHAR_BACKWARD denepding on isRTL() for 'logical' movement. Andre'
I'm not sure that this is correct: precisely in the situations we're discussing here, of LTR embedded in RTL, we want the LEFT key to keep moving forward, even if forward means right...
The truth is, there's really no way out: there'll always be situations in which the movement that we want will not correspond to the key pressed (unless we implement strictly visual cursor movement, but that has it's own problems --- see the note in bug 3551), so I don't know that there's any point in changing the LFUN's name...
Dov
