Review Request: Move suggestions menu for misspelled word on top.

Thomas McGuire mcguire at kde.org
Wed Nov 11 15:07:04 GMT 2009


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

Ship it!


Looks good to me, please commit.

- Thomas


On 2009-11-10 09:04:30, Harsh J wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/2097/
> -----------------------------------------------------------
> 
> (Updated 2009-11-10 09:04:30)
> 
> 
> Review request for kdelibs.
> 
> 
> Summary
> -------
> 
> Minor fix for the Bug 194979.
> 
> Bug-report requests that "Suggestions" in the context menu of spell-checker for a word, be moved to top instead of last.
> 
> Re-ordered the menu/action-adding code in KTextEdit to make the spelling "suggestions" appear first.
> 
> (Diff 1 keeps them in a popup, diff 2 moves them to top level - as in screenshots)
> 
> 
> This addresses bug 194979.
>     https://bugs.kde.org/show_bug.cgi?id=194979
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdelibs/kdeui/widgets/ktextedit.cpp 1046992 
> 
> Diff: http://reviewboard.kde.org/r/2097/diff
> 
> 
> Testing
> -------
> 
> Recompiled kdelibs and the superficial change appears fine.
> 
> 
> Screenshots
> -----------
> 
> Moved to top entry - suggestions
>   http://reviewboard.kde.org/r/2097/s/256/
> Moved to top level - suggestions
>   http://reviewboard.kde.org/r/2097/s/257/
> 
> 
> Thanks,
> 
> Harsh
> 
>





More information about the kde-core-devel mailing list