Change toolbar background color from IFrame API #16468 fixed

[vishal2005025]

📝 Description:

This PR adds support for dynamically setting the toolbar background color through the IFrame API, improving customizability for embedded integrations.

🚀 Changes Introduced:

• Added a new IFrame API command:

  api.executeCommand('setToolbarBackgroundColor', '#ff0000');

[jitsi-jenkins]

Hi, thanks for your contribution!
If you haven't already done so, could you please make sure you sign our CLA (https://jitsi.org/icla for individuals and https://jitsi.org/ccla for corporations)? We would unfortunately be unable to merge your patch unless we have that piece :(.

[damencho]

There is no point of having a special command for this when you can use overwriteConfig command or you can pass it as a configOverwrite when initializing the iframAPI.
You added backgroundColor to toolbarConfig in config type, but you missed adding it to the config.js file where we put all configs with the default values they are initialized:

jitsi-meet/config.js

Line 895 in 57865d7

// toolbarConfig: {

When adding a config, that should be implemented for both web and mobile.
Drop the not relevant changes as the gitignore one.

[vishal2005025]

Got it, thanks for the clarification!
I’ll switch to using overwriteConfig instead of adding a new command and make sure backgroundColor is added to config.js with proper defaults for both web and mobile.
Also dropping the unnecessary .gitignore change.
I’ll update the PR shortly!

[vishal2005025]

Updated the implementation to use configOverwrite for toolbar background color on web and mobile. Previous review comments have been addressed - please review.

[saghul]

Please don't uncomment this, leave it commented out, reflecting the defaults.

[vishal2005025]

Thanks for pointing that out! I’ve fixed it- it’s now left commented out as per the defaults.

[saghul]

Add a trailing comma please.

[vishal2005025]

Thanks! I have added the trailing comma as suggested