Set Tabulator to render into explicit popup container
Not entirely happy with this as the editor isn't aligned quite right but I played with all the options and couldn't find one that produced the correct result.
Fixes https://github.com/holoviz/panel/issues/7295
Codecov Report
:white_check_mark: All modified and coverable lines are covered by tests.
:white_check_mark: Project coverage is 87.37%. Comparing base (977e801) to head (d09c0cd).
:warning: Report is 1 commits behind head on main.
Additional details and impacted files
@@ Coverage Diff @@
## main #7299 +/- ##
==========================================
- Coverage 87.40% 87.37% -0.04%
==========================================
Files 347 347
Lines 53618 53618
==========================================
- Hits 46867 46850 -17
- Misses 6751 6768 +17
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
:rocket: New features to boost your workflow:
- :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
- :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
Seeing this problem for one of my examples in the original issue:
Yep, seeing that too. Not sure what to do about it.
What about making this setting an option for now?
Or maybe document the behavior, as it can be enabled by the configuration directly: pn.widgets.Tabulator(... configuration={"popupContainer": True})
So I'm leaning towards merging this as is now. It turns out the weird offset issue occurs only for the last column and in the meantime this PR also fixes:
https://github.com/holoviz/panel/issues/8096
and the coloring of the popup.
Okay, fixed the offset issue by inserting an empty last column.