Review Request 111608: Fix "truncated Details View header" when using a non-Oxygen style

[Christoph Feck]

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


For whatever reason, it does not affect Skulpture style. You sure can commit it, while I try to find out if the issue is with Skulpture or the header painting.

- Christoph Feck


On July 19, 2013, 6:12 p.m., Frank Reininghaus wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111608/
> -----------------------------------------------------------
> 
> (Updated July 19, 2013, 6:12 p.m.)
> 
> 
> Review request for Dolphin.
> 
> 
> Description
> -------
> 
> When using a non-Oxygen style, and the widths of the columns in Details View are too small to fill the entire width of the view, the headers look "truncated" (see screen shot in the bug report). This does not happen with Oxygen - an empty header is shown instead of the white empty space.
> 
> I looked at the code and found that the painting of the "empty header" is inconsistent with the other headers, which is probably the reason why the other styles get confused:
> 
> a) No QStyleOptionHeader is used
> b) Even if an empty header must be drawn, the last column is drawn with the option QStyleOptionHeader::End.
> 
> I propose to change this by moving the code for painting the empty header to KItemListHeaderWidget::paintRole(): when the last column is drawn, it checks if there is some space left that needs to be filled and uses QStyleOptionHeader::Middle for the last column then (I don't see another way how issue b) could be solved cleanly).
> 
> 
> This addresses bug 301800.
>     http://bugs.kde.org/show_bug.cgi?id=301800
> 
> 
> Diffs
> -----
> 
>   dolphin/src/kitemviews/private/kitemlistheaderwidget.cpp 0f1f20b 
> 
> Diff: http://git.reviewboard.kde.org/r/111608/diff/
> 
> 
> Testing
> -------
> 
> Works for me with Oxygen and Plastique.
> 
> 
> Thanks,
> 
> Frank Reininghaus
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20130721/dec3e6ee/attachment.htm>