Codejock Developer Community : Yet another bug: Ribbon, SetCurSel

oleg wrote:

pTab = pControl->FindTab(ID_TAB_CHARTFORMAT);

BOOL bOldVisible = pTab->IsVisible();
pTab->SetVisible(bVisible);

if (bVisible && !bOldVisible)
{
pTab->Select();
}

it will activate it only once when it first become visible and resolve stack overflow problem

That workaround works, but it's not perfect. If I call SetCurSel in the update handler the update handler is beeing called again but GetCurSel still returns the old index and not the one I've set before.

]]> Fri, 04 Aug 2006 09:32:28 +0000 http://forum.codejock.com/forum_posts.asp?TID=4730&PID=14680&title=yet-another-bug-ribbon-setcursel#14680 <![CDATA[Yet another bug: Ribbon, SetCurSel : Hi, This is not good idea toselect...]]> http://forum.codejock.com/forum_posts.asp?TID=4730&PID=14674&title=yet-another-bug-ribbon-setcursel#14674 Author: Oleg
Subject: 4730
Posted: 04 August 2006 at 8:59am

Hi,

This is not good idea to select tab each time in update handler - user will not be able to switch tabs. I'd change it to

pTab = pControl->FindTab(ID_TAB_CHARTFORMAT);

BOOL bOldVisible = pTab->IsVisible();
pTab->SetVisible(bVisible);

if (bVisible && !bOldVisible)
{
pTab->Select();
}

it will activate it only once when it first become visible and resolve stack overflow problem

]]> Fri, 04 Aug 2006 08:59:33 +0000 http://forum.codejock.com/forum_posts.asp?TID=4730&PID=14674&title=yet-another-bug-ribbon-setcursel#14674 <![CDATA[Yet another bug: Ribbon, SetCurSel : The problem is in CXTPRibbonBar::RebuildControls...]]> http://forum.codejock.com/forum_posts.asp?TID=4730&PID=14673&title=yet-another-bug-ribbon-setcursel#14673 Author: ABuenger
Subject: 4730
Posted: 04 August 2006 at 8:56am

The problem is in CXTPRibbonBar::RebuildControls where now OnIdleUpdateCmdUI is beeing called.

Is there any reason why OnIdleUpdateCmdUI is beeing called explicitly?

]]> Fri, 04 Aug 2006 08:56:40 +0000 http://forum.codejock.com/forum_posts.asp?TID=4730&PID=14673&title=yet-another-bug-ribbon-setcursel#14673 <![CDATA[Yet another bug: Ribbon, SetCurSel : In the RibbonSample add the following...]]> http://forum.codejock.com/forum_posts.asp?TID=4730&PID=14657&title=yet-another-bug-ribbon-setcursel#14657 Author: ABuenger
Subject: 4730
Posted: 03 August 2006 at 4:59pm

In the RibbonSample add the following lines to CRibbonSampleView::OnUpdateRibbonTab at the end of the method:

...

pTab = pControl->FindTab(ID_TAB_CHARTFORMAT);
pTab->SetVisible(bVisible);

if (bVisible)
{
pTab->GetRibbonBar()->SetCurSel(7);
}

Then start the app and click on Insert > Chart....

That makes it impossible to automatically select the first visible contextual tab like in Office 2007.

I've the same problem in my app and it worked with 10.20

]]> Thu, 03 Aug 2006 16:59:13 +0000 http://forum.codejock.com/forum_posts.asp?TID=4730&PID=14657&title=yet-another-bug-ribbon-setcursel#14657