I'm trying to duplicate the functionality of how many applications handle tabs
where the last tab is treated as a special case and is used solely to create
new tabs. So when my widget launches it looks like:
\ Tab 1 /\ + /
With "Tab 1" set as the current index. Left clicking on the '+' will add a new
tab, "Tab 2", and sets "Tab 2" as the current index, like so:
\ Tab 1 /\ Tab 2 /\ + /
To achieve this, I've connected the currentChanged(int index) signal to a slot
that checks to see if the current index is the last index, and if so it adds a
new tab, and sets the newly created one as current. If any other tab is the
current index, the slot does nothing. Here's the code for the slot:
void customTabWidget::slotTabBarClicked(int index)
{
if (index != (count() - 1))
{
// if it's not the last one, we don't need to do anything
return;
}
// if it's the last tab, we add a new tab
slotAddTab();
}
This works great when the user left-clicks on the '+' tab.
I've also added a contextual menu that allows the user to rename or delete a
tab by right clicking one of the tabs. I'm handling that in mousePressEvent().
This functionality should work for all tabs EXCEPT the final tab; the user
should never be able to rename or delete the '+' tab, since it's a special
case. In mousePressEvent(), I'm attempting to detect which index the user
right clicked on so that I can rename or delete the correct tab. Here's the
code for mousePressEvent:
void customTabWidget::mousePressEvent(QMouseEvent *event)
{
QTabWidget::mousePressEvent(event);
if (event->button() == Qt::RightButton)
{
for (int i=0; i < tabBar()->count()-1; i++)
{
QPoint clickPos = mapToGlobal(event->pos());
if
(tabBar()->tabRect(i).contains(tabBar()->mapFromGlobal(clickPos)))
{
qDebug("Tab count %d, right click on %d", tabBar()->count(), i);
setCurrentIndex(i);
slotShowContextualMenu(clickPos);
}
}
}
}
Since there's no reason to show that contextual menu on the '+' tab, I'd like
to disable the right click functionality just on the last tab, which I'm trying
to do in the mousePressEvent for loop, by only looping to bar->count() - 1.
But the problem I'm having is that right-clicking on the '+' tab emits
currentChanged(int) BEFORE mousePressEvent() is called, so a new tab is being
created in slotTabBarClicked(), so before the user right clicks, count() is 2,
current index is 0, and the '+' is on index 1. Once the user right clicks on
the '+' tab, by the time we get to mousePressEvent(), count() is 3, current
index is 1, and the '+' tab is at index 2. So the mapping doesn't work and I
end up displaying a contextual menu on the newly created "Tab 2", even though
the user right-clicked on the '+' tab.
The behavior I want is right-clicking on the '+' should do nothing; it
shouldn't add a new tab, it shouldn't show a contextual menu.
What am I missing? Or how should I be trying to do this? If there was a
separate signal for left vs. right clicks, it'd be easy. But I don't see a way
to easily detect in slotTabBarClicked(int) that it was a right-click that
caused currentChanged(int) to be emitted...
Sean
_______________________________________________
Interest mailing list
Interest@qt-project.org
http://lists.qt-project.org/mailman/listinfo/interest
