QTabWidget, setting colour of tab itself ?

SPlatten

@JoeCFD , unfortunately its completely out of my control and the company using it won't be updating it anytime soon as there are other dependencies / reasons why they can't.

JoeCFD

@SPlatten You may push them for an update and get more contracts.

SPlatten

@JoeCFD , the company is a world wide provider of defence and weapon tech....they're clients are governments and the equipment is years out of date, again, I have requested an updated, but I can't do anything about it.

JoeCFD

4.8 qtabbar_cpp source code is here. copy line 1569 to 1648 to your paint event. Not sure if this will work.
https://dreamswork.github.io/qt4/qtabbar_8cpp_source.html
painting of tabs is done at line 1622.
Look at lines 1594 and 1595. These lines call initStyleOption() to create QStyleOptionTabV3 tab for each tab which is a subclass of QStyleOption
https://dreamswork.github.io/qt4/classQTabBar.html#a41b394d892263b6b5a0705fb979f3c8e
and each QStyleOption has a public palette with background color. You can change it. Try to print out the background color to see what it is.
you can find QStyleOptionTabV3 and QStyleOptionTab here
https://dreamswork.github.io/qt4/qstyleoption_8h_source.html
This way may be simpler. Good luck.

JonB

@SPlatten
Without wanting to digress this thread. I am surprised given the age of an old 4.8 implementation they now insist they want a change in a tab colour! Can't you say "Sorry, I have looked into it, this old version won't let me do that" ?

SPlatten

@JoeCFD , unfortunately the company firewall will not allow access to the dreamswork domain.

SPlatten

@JonB , I was brought into this company 5 months ago and the project is very old, there is a lot of issues to resolve and this is just one of many.

SPlatten

@J.Hilk, @JonB , @JoeCFD, I tried another tack...that is calling tabButton of QTabBar to get the tab widget, I was then going to try setting the widget style sheet, however failed at the first hurdle:

int intTabs(count()); //This returns 10, which is the number of tabs
for( int intTab=0; intTab<intTabs; intTab++ ) {
    QString strText(tabText(intTab));  //This works and gets the tab text, e.g. &Overall
    QWidget* pobjTab(tabButton(intTab, QTabBar::LeftSide));
    if ( !pobjTab ) {
        pobjTab = tabButton(intTab, QTabBar::RightSide);
    }
}

Calling tabButton with QTabBar::LeftSide or QTabBar::RightSide returns NULL. Have I done something wrong or is this something else that doesn't work in 4.8 ?

SPlatten

@JoeCFD , just tried to access:
https://dreamworks.github.io/qt4/qtabbar_8cpp_source.html

on my iPhone and get 404, sorry must be my fat fingers, closed browser and tried again this time ok.

JonB

@SPlatten said in QTabWidget, setting colour of tab itself ?:

Calling tabButton with QTabBar::LeftSide or QTabBar::RightSide returns NULL. Have I done something wrong or is this something else that doesn't work in 4.8 ?

I believe either (a) the QTabBar does not have any QWidgets on/for its tabs at the time you call this or (b) it never has any QWidget on/for its tabs unless you call QTabBar::setTabButton(int index, QTabBar::ButtonPosition position, QWidget *widget).

SPlatten

@JoeCFD , QStyleOptionTabV2 and QStyleOptionTabV3 are according to official documentation obsolete and replaced with QStyleOptionTab.

SPlatten

@JonB , (b) is the case, I was hoping that each button on the tab bar was a widget available via this call.

JonB

@SPlatten
I think/suspect not. By default it does not use/place any QWidget on/for the tabs, rather something internal (just the text on the tab, no special widget).

Mind you, if it helps your case (I don't know if it does) you could explicitly use QTabBar::setTabButton() to put your own widget there, then you can access that widget. I don't know whether that would aid you in your issue.

You were referred to https://stackoverflow.com/questions/46137500/qt-tabwidget-each-tab-title-background-color. There in answer https://stackoverflow.com/a/46139321/489865 you are shown a picture with the tabs having their own (different) colors. Could you state clearly and unequivocally whether (a) this is what you want and (b) (if so) why you can't use that solution?

SPlatten

@JonB , in my code I am now writing the geometry of each tab when each is selected to the log file and I can see the geometry is correct , I can also see that the selection of the correct colour is chosen, but is this code correct:

opt.palette.setColor(QPalette::Button, QColor(kErrorColour));

opt is obtained from a call to:

initStyleOption(&opt, intTab);

Where intTab is 0 to N and opt is defined as QStyleOptionTab.

JonB

@SPlatten
Does not seem unreasonable. Would not work if (a) kErrorColour's value is not acceptable to QColor or (b) you are using a stylesheet on the element, as stylesheets override palette changes.

Have you first tried the code shown in that SO solution? Does that work for you as they show? Then you can fiddle with changing that to whatever it is you want till it works or goes wrong.

SPlatten

@JonB , kErrorColour:

const char_t kErrorColour[] = "red";

JonB

@JonB said in QTabWidget, setting colour of tab itself ?:

Have you first tried the code shown in that SO solution? Does that work for you as they show? Then you can fiddle with changing that to whatever it is you want till it works or goes wrong.

SPlatten

@JonB , thats the code my class is based on and no I don't have any of the tabs changing colour.

JonB

@SPlatten said in QTabWidget, setting colour of tab itself ?:

thats the code my class is based on

Why won't you try their code instead of yours? How do we know whether something you have done has affected the outcome? Why do we have to guess whether that might be the case or not when you could try a solution shown to have been working? First try their exact code, then report whether it works for you or not.....

SPlatten

@JonB , mainly because its a lot of backtracking and changes that to be honest its simple enough to see what this is doing and mine does almost exactly the same.

What I will do instead is create a simple application using Qt Creator and 4.8 with the original code to see if that works.