fix(rows): let file picker and smart picker open above row modals #2515 - #2912
Conversation
The row modals overrode .modal-mask to z-index 9999, one level above the NcModal default of 9998. Since NcModal teleports to body, pickers opened from a rich text cell shared that stacking context and always painted underneath. CreateRow.vue carried the same override as EditRow.vue. Fixes nextcloud#2515 Assisted-by: ClaudeCode:claude-opus-5 Signed-off-by: foolfitz <foolfitz@gmail.com>
|
Did you try different use case ? see #2126 |
enjeck
left a comment
enjeck
left a comment
There was a problem hiding this comment.
what Aveyron-RetD said above. Please attach screenshots showing the various cases.
Right now, i think we have a "deadlock" where if we have this, we're sacrificing the other
|
For me this bug is not present on 33.0.6 tables app 2.2.1 |
|
I made some screenshots. Tables editing
Embed a nextcloud table in Text
|
|
@foolfitz thanks for the screenshots. What about after the table is embedded into Text and you edit the row? Does the pop up sit right? |
|
@enjeck Do you mean these? |
|
@foolfitz The AI Policy CI is failing. Can you please sign your commits with it e.g |
|
@enjeck 
I went back through the AI Policy run, and the failure happens at the "Label PR as AI assisted" step — the trailer check itself passed: 
I talked it through with Claude, and it looks like the blocker is on the GitHub Actions side rather than in the commit message. |
3 participants
The row modals overrode .modal-mask to z-index
9999, one level above the NcModal default of9998. Since NcModal teleports to body, pickers opened from a rich text cell shared that stacking context and always painted underneath.CreateRow.vuecarried the same override asEditRow.vue.Fixes #2515
Assisted-by: ClaudeCode:claude-opus-5
🤖 AI (if applicable)