Hi, Sorry for all the trouble. Sho on irc asked me that he want to test the ibus 1.5 support easier so I get it merged for kde4. Unfortunately there's no further kdeplamsa-addons for kde4 and I didn't finish this before 4.14, so it's also either leave it broken in kde4 or add a usable version.
This change doesn't break old things but fix a never supported version for ibus, and the old support is also kept in a different directory. So there is no actual code deletion, and doesn't change the behavior for old version of ibus 1.4. The line number looks huge for following reason: 1. The ibus upstream changed quite a lot and made that ibus support must parse a gtk key string. It's not possible to use gtk for just a keystring parse so I was forced to copy some gdk key string parsing code here. In this part of code there is a hardcoded key symbol to string table which is unfortunately huge. 2. The old supporting code is also kept but moved, and new code is also based on those code. For i18n, there is no new string should be introduced. In summary, this change doesn't break on old version, and introduce a fix of supoort for new version which is never supported before. On Thu, Oct 2, 2014 at 3:52 PM, Albert Astals Cid <aa...@kde.org> wrote: > Hi Weng, I have been just told you just merged your > origin/xuetianweng/kimpanel-ibus-1.5 > into > KDE/4.14 > in the kdeplasma-addons repo > > That brings > 27 files changed, 10677 insertions(+), 1424 deletions(-) > > Has this been discussed anywhere? Because honestly this seems quite a big > change for a stable branch. > > Cheers, > Albert >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
