Skip to content

Commit 298e84b

Browse files
authored
Warn-colour the previous-workflow divergence note; widen the commit selector's hint gap (#770)
1 parent ded438b commit 298e84b

6 files changed

Lines changed: 29 additions & 4 deletions

File tree

Cargo.lock

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

quilt-sync/CHANGELOG.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,12 @@
99
<!-- markdownlint-disable MD013 -->
1010
# Changelog
1111

12+
## [v0.18.3-alpha16] - 2026-07-14
13+
14+
### Changed
15+
16+
- In the commit dialog, the note that the previous revision used a different workflow is now shown in a warning colour (so QuiltSync's default-wins preselection can't silently switch the workflow unnoticed), with a little more spacing below the selector (<https://github.com/quiltdata/quilt-rs/pull/770>)
17+
1218
## [v0.18.3-alpha15] - 2026-07-13
1319

1420
### Fixed

quilt-sync/src-tauri/Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
[package]
22
name = "quilt-sync"
3-
version = "0.18.3-alpha15"
3+
version = "0.18.3-alpha16"
44
authors = ["Quilt Data, Inc."]
55
description = "Cross-platform desktop application for editing Quilt data packages"
66
documentation = "https://docs.quiltdata.com"

quilt-sync/ui/assets/css/components/workflow_select.css

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,14 @@
5454
margin: var(--q-ui-size-0-5) 0 0;
5555
}
5656

57+
/* The previous-revision divergence note: warning-coloured so a silent switch
58+
* away from the previous pick (QuiltSync preselects the bucket default) is not
59+
* missed. Text stays factual; only the colour signals attention. Declared
60+
* after `.qui-workflow-hint` so it wins the colour at equal specificity. */
61+
.qui-workflow-note-warn {
62+
color: var(--q-ui-palette-warning);
63+
}
64+
5765
/* Malformed-config notice. Normal flow (not absolute) so a long, multi-line
5866
* schema-violation reason claims its own vertical space and wraps, instead of
5967
* overlapping the field above it. Mirrors `.qui-workflow-hint`, error-coloured. */

quilt-sync/ui/assets/css/pages/commit.css

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,12 @@
3232
gap: var(--q-ui-size-4);
3333
}
3434

35+
/* A bigger gap between the selector box and its hint text below it (the
36+
* divergence note and the "Open in catalog" links). */
37+
.qui-workflow-field {
38+
margin-bottom: var(--q-ui-size-1);
39+
}
40+
3541
/* Keeps a field and its advisory violation together as one unit, so the
3642
* form's inter-field `gap` falls between groups, not between a field and its
3743
* own error. */

quilt-sync/ui/src/components/workflow_select.rs

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -566,10 +566,15 @@ fn workflow_dropdown(
566566
<Show when=show_required_hint>
567567
<span class="qui-workflow-error">"Workflow is required for this bucket."</span>
568568
</Show>
569-
// Neutral note: appears when the current pick diverges from the
569+
// Divergence note: appears when the current pick differs from the
570570
// previous revision's stamp, and disappears when they match again.
571+
// Warning-coloured — QuiltSync's default-wins preselection can
572+
// silently switch the workflow away from the previous pick, so the
573+
// divergence is flagged rather than whispered.
571574
{move || {
572-
note.get().map(|text| view! { <p class="qui-workflow-hint">{text}</p> })
575+
note.get().map(|text| view! {
576+
<p class="qui-workflow-hint qui-workflow-note-warn">{text}</p>
577+
})
573578
}}
574579
// Catalog links for the current selection (config.yml + the
575580
// selected workflow's schemas), recomputed as the selection changes.

0 commit comments

Comments
 (0)