fixed the Table background color when inverted and basic active at same time

[lord2anil]

closes #4603

[netlify]

✅ Deploy Preview for volto canceled.

Name	Link
🔨 Latest commit	2f3da83
🔍 Latest deploy log	https://app.netlify.com/sites/volto/deploys/641d47b17bc75f00084bca4c

[stevepiercy]

@lord2anil can you open a PR on a branch in the volto repo? GitHub Actions will only run on branches inside the Plone organization. @fredvd @sneridagh would you please take a look at permissions for @lord2anil to open a PR in volto? We might need to make an adjustment.

[fredvd]

@stevepiercy It's primarily a workflow issue I think. The towncrier checks for example have run for lord2anil's Pull Request from his own workspace, because the on: definition explicitly mentions pull_request.

https://github.com/plone/volto/actions/runs/4508573482

The other Volto workflows are registered on push: for a branch on the main repo. @sneridagh We need to take a look at the workflows so that the necessary ones like unit tests are also running on pull requests. .

[netlify]

✅ Deploy Preview for plone-components canceled.

Name	Link
🔨 Latest commit	cb84415
🔍 Latest deploy log	https://app.netlify.com/sites/plone-components/deploys/66a2b823551f080008a2802a

[JeffersonBledsoe]

@lord2anil Thanks for the contribution! On testing this PR, I've noticed a couple of issues:

• The 'black' colour doesn't match that of 'inverted'. This could cause an issue if the themeing system has been used to change the colour of 'inverted'.
  • I would suggest solving this in a .less file while you can check for .inverted.basic and read from the @invertedBackground and other inverted table variables
• The current fix only applies to edit mode and so it breaks again in view mode. The above suggestion about using the .less files should solve this.

Are you able to make these changes and we can work towards getting this PR merged?

[stevepiercy]

Suggested change

	fixed the Table background color when inverted and basic active at same time @lord2anil
	Fixed the table background color when its `inverted` and `basic` styles are active at the same time. @lord2anil

[lord2anil]

Thanks for the feedback. Due to time constraints, I cannot implement the changes right now.

[ichim-david]

The fix isn't what is needed. Due to the time constraints of the author I am closing this pull request, we can open a new one with the solution proposed by @JeffersonBledsoe