Review Request: Kexi Tables/Forms: show indication that edited or pasted text is trimmed

Jarosław Staniek staniek at kde.org
Wed Nov 7 20:59:17 GMT 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107068/
-----------------------------------------------------------

(Updated Nov. 7, 2012, 8:59 p.m.)


Review request for Calligra, Adam Pigg, Dimitrios Tanis, Radosław Wicik, and Oleg Kukharchuk.


Changes
-------

As proposed by Dimitrios:
- display different the warning
- while user is editing, hide the warning if overflow is fixed
- upon saving, trim if needed


Also:
- the warning now informs how many characters would be trimmed (so is updated on every editing action)
- refactored code displaying warning messages to avoid duplicates


Description
-------

Tables/Forms: show indication that edited or pasted text is trimmed.

See also http://forum.kde.org/viewtopic.php?f=221&t=102475 thread.

Screencasts with the feature:
http://www.youtube.com/watch?v=Qdd7CaBonq0
http://www.youtube.com/watch?v=wjyGsWxnAEs


This addresses bug 301136.
    http://bugs.kde.org/show_bug.cgi?id=301136


Diffs (updated)
-----

  kexi/core/kexidataiteminterface.h 639e875 
  kexi/core/kexidataiteminterface.cpp 8eb9d4d 
  kexi/plugins/forms/kexiformscrollview.h 43ce3f9 
  kexi/plugins/forms/kexiformscrollview.cpp 9ba0fc4 
  kexi/plugins/forms/widgets/kexidblineedit.h faf954b 
  kexi/plugins/forms/widgets/kexidblineedit.cpp f023937 
  kexi/widget/dataviewcommon/kexidataawareobjectiface.h ef5a892 
  kexi/widget/dataviewcommon/kexidataawareobjectiface.cpp ba5266c 
  kexi/widget/tableview/kexiinputtableedit.h fe20fb1 
  kexi/widget/tableview/kexiinputtableedit.cpp 8c62a28 
  kexi/widget/tableview/kexitableview.h e01ba05 
  kexi/widget/tableview/kexitableview.cpp a0e17c6 
  kexi/widget/tableview/kexitextformatter.h 880a36c 
  kexi/widget/tableview/kexitextformatter.cpp c119056 

Diff: http://git.reviewboard.kde.org/r/107068/diff/


Testing
-------

Properly displays warnings and trims data


Thanks,

Jarosław Staniek

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20121107/6aa3b43f/attachment.htm>


More information about the calligra-devel mailing list