Review Request 115601: [parley][PATCH] Fixes the editor dock widgets leaking out of screen and becoming inaccessible bug, and window resizing issue with multiple widgets

[Albert Astals Cid]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/115601/#review49744
-----------------------------------------------------------



src/editor/editor.cpp
<https://git.reviewboard.kde.org/r/115601/#comment34980>

    "this->" is usually frowned upon amongst C++ coders, just don't put it there.


- Albert Astals Cid


On Feb. 11, 2014, 7:40 p.m., Amarvir Singh wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/115601/
> -----------------------------------------------------------
> 
> (Updated Feb. 11, 2014, 7:40 p.m.)
> 
> 
> Review request for KDE Edu.
> 
> 
> Repository: parley
> 
> 
> Description
> -------
> 
> Fixes the bug that caused widgets in the editor to leak out of the bottom of the screen, if multiple widgets were selected in view, making them inaccessible.
> The bug didn't allow the screen to be resized either as the dock widgets could not be made smaller.
> This made it an annoying bug, as it didn't allow the user to simply resize the window to access the cutoff widgets. They had to be closed, and the window had to be minimized-maximized
> to restructure the dock widgets.
> 
> Steps to reproduce: 
> 1. Go to editor mode.
> 2. View->add multiple widgets (eg synonyms, antonyms, image, latex, etc.)
> 3. The widgets start leaking out even in a Full HD screen, and some, hence, become inaccessible.
> 4. Try resizing window (decreasing size from the top), doesnt work, as QDockWidgets take a defined space.
> 
> Expected Result: 
> Widgets should not be cutoff, and resizing should be possible.
> 
> This commit makes each widget scrollable so that they can be reduced in size, to allow the window to be made smaller. Also they are tabbed together, to ensure widgets don't leak out,
> even for lower resolutions, and it also ensures that they remain accessible.
> 
> PS: To properly check for the changes in the dockwidgets after the patch, be sure to not use the configuration saved, rename it or delete it, or just rename kdehome directory. This is because dockwidget positions and layout is saved. Then continue adding all widgets.
> 
> 
> Diffs
> -----
> 
>   src/editor/editor.cpp f3787a3 
> 
> Diff: https://git.reviewboard.kde.org/r/115601/diff/
> 
> 
> Testing
> -------
> 
> Tested, from 1920x1080 to 800x600 resolutions. 
> 
> 
> Thanks,
> 
> Amarvir Singh
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20140213/45db5eb7/attachment.html>