[PATCH] wpa_gui-qt4: silence compile warning + wpsTab enable/disable
Kel Modderman
kel
Sun Feb 1 12:33:50 PST 2009
On Monday 02 February 2009 05:58:55 Jouni Malinen wrote:
> On Mon, Feb 02, 2009 at 03:51:49AM +1000, Kel Modderman wrote:
>
> > Is this better?
>
> > - wpsTab->setEnabled(types.contains("WSC"));
> > + wpaguiTab->setTabEnabled(wpaguiTab->indexOf(wpsTab),
> > + types.contains("WSC"));
>
> Interesting.. It looks like the wpsTab->setEnabled(FALSE) grays out the
> contents of the tab, but allows the tab to be selected while
> wpaguiTab->setTabEnabled(idx, FALSE) grays out the tab selection text
> and does not allow the tab to be selected. However, if the tab gets
> selected somehow in the program, the contents of the tab is not grayed
> out.. It looks like we actually should add both of these.
Yep, agreed.
>
> removeTab() could be even better from the UI clarity view point, but it
> would require more work to bring back the tab if needed.
>
Agreed again.
Thanks for committing a nicer patch.
Kel.
More information about the Hostap
mailing list