Skip to content

Mutation editor#125

Merged
oliver-sanders merged 2 commits intooliver-sanders:2390from
MetRonnie:mutation-editor
Jan 7, 2026
Merged

Mutation editor#125
oliver-sanders merged 2 commits intooliver-sanders:2390from
MetRonnie:mutation-editor

Conversation

@MetRonnie
Copy link

No description provided.

}

.c-mutation {
.c-mutation-menu-item {
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This class name was changed leading to the icons in the menu to be lower opacity without this style

Comment on lines -327 to +330
<style scoped>
<style scoped lang="scss">
Copy link
Author

@MetRonnie MetRonnie Jan 7, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nested CSS does not yet have Baseline support: https://caniuse.com/css-nesting, can fix this by specifying SCSS

<v-card class="c-mutation">
<v-card
class="c-mutation"
variant="flat"
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The card had unwanted shadows when in the lumino tab view unless set to the flat variant

Comment on lines +147 to 150
layoutWatcher.pause()
// Register with Lumino that the dock panel is no longer used,
// otherwise uncaught errors can occur when restoring layout
dockPanel.dispose()
Copy link
Author

@MetRonnie MetRonnie Jan 7, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't quite get to the bottom of cylc#2394 (comment), but the layout watcher was for some reason being triggered after the dock panel was disposed. Not sure why, I investigated if it was due to the FormGenerator changing its parent v-model but am not convinced that's the cause. At least I think it makes sense to stop the layout watcher before disposing of the dock panel.

@oliver-sanders oliver-sanders merged commit ddbe741 into oliver-sanders:2390 Jan 7, 2026
5 checks passed
@MetRonnie MetRonnie deleted the mutation-editor branch January 7, 2026 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants