[16.0][FIX] web_responsive: Remove overflow:visible to tables to avoid side effects

[victoralmau]

Remove overflow:visible to tables to avoid side effects

Without web_responsive

With web_responsive - before

With web_responsive - after

Please @CarlosRoca13 can you review it?

@Tecnativa TT50468

[OCA-git-bot]

Hi @SplashS, @Tardo,
some modules you are maintaining are being modified, check this out!

[pedrobaeza]

Please check why that was introduced, as maybe removing it has another "side effect". You can ask as well the original contributor.

[victoralmau]

These changes were added at #2405 (I don't know exactly why) and removed at #2684.

Ping @bvanjeelharia Can you tell us why that code was added, is this change correct and will not cause side effects or do you propose a better one? Thanks

[pedrobaeza]

The ping wasn't correct. Trying again @bvanjeelharia

[pedrobaeza]

If removed in 17.0, it seems it's not very needed.

[victoralmau]

Specifically, this piece of code was removed in v17 https://github.com/OCA/web/blob/16.0/web_responsive/static/src/legacy/scss/web_responsive.scss#L250-L288

[pedrobaeza]

Well, it was removed because Odoo incorporates the sticky feature natively, so maybe the overflow was not removed specifically, but inside that block. Anyway, it looks more and more a side effect.

[CarlosRoca13]

/ocabot merge patch

[OCA-git-bot]

This PR looks fantastic, let's merge it!
Prepared branch 16.0-ocabot-merge-pr-2910-by-CarlosRoca13-bump-patch, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at b9fd7f3. Thanks a lot for contributing to OCA. ❤️