[Kde-pim] Review Request: Remove close button on the Article tab in Akregator

[Laurent Montel]

> On July 10, 2012, 8:13 p.m., Allen Winter wrote:
> > I tested this patch and I don't like the new behavior.
> > 
> > For me, the first close button on the first tab disappears immediately after I open another tab, for example with I click on the "The complete story" link.
> > 
> > As long as more than 1 tab is open, I think the close button should be available on all tabs.
> 
> Phương Lê Hoàng wrote:
>     You can read the bug 173067. The Article tab is the first tab and it can not be repositioned to second or so on. In the bug 173067, ostefield has disabled the close function for the Article tab, but he didn't remove the close button altogether with its function. What I want to do is to completely remove that close button, so users won't get confused about whether that tab can be closed or not.

If button is disable is enough.
And if Allen tested it and see "For me, the first close button on the first tab disappears immediately after I open another tab" it's a new bug.
So for me this patch is not good (Ok not tested yet, but by default disable button is enough)


- Laurent


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


On July 10, 2012, 1:58 p.m., Phương Lê Hoàng wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105504/
> -----------------------------------------------------------
> 
> (Updated July 10, 2012, 1:58 p.m.)
> 
> 
> Review request for KDEPIM.
> 
> 
> Description
> -------
> 
> Completely remove the close button on the Article tab in Akregator.
> Related bug: https://bugs.kde.org/show_bug.cgi?id=173067 (This bug is marked as solved).
> 
> 
> Diffs
> -----
> 
>   akregator/src/tabwidget.cpp 82aa5bb 
> 
> Diff: http://git.reviewboard.kde.org/r/105504/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Phương Lê Hoàng
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/