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

[Phương Lê Hoàng]

> 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.
> 
> Laurent Montel wrote:
>     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)

What Allen said is the wanted behaviour. The Article tab (aka the first tab) is the default tab and it can not be closed. Keeping the close button there is unnecessary and causes confusing for users. This duplicated bug 171360 explains what the users want. Disable button is not enough because it still can be clicked, though after clicking, nothing happens.


- Phương


-----------------------------------------------------------
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/