feat(studio): brand-agnostic onboarding hints + clickable checklist + theme-picker clarity
Some checks failed
Security / Security check (push) Failing after 2s
Some checks failed
Security / Security check (push) Failing after 2s
Three changes:
1. Onboarding checklist hints no longer mention DeepWiki,
Semgrep, text.extract, or extract-summarize.yaml by
name. Brand- and file-specific copy moved to neutral
feature pointers ('add an MCP source', 'install a
module'). Brand names belong on the welcome page only
if they're load-bearing for understanding, which they
aren't.
2. Checklist rows are now clickable. Tapping a row navigates
to the page where the operator can complete it — MCP +
AI open Settings, Module opens Store, Flow opens Flows.
StudioShellState gains a public navigateTo(pageId) helper
so descendants can drive sidebar selection without
plumbing a controller through props.
3. Theme picker hint copy rewritten. Old hint suggested
'Custom' was a single-colour tweak; new copy spells out
that Material 3 derives a full palette (primary,
secondary, tertiary, surface, …) from one seed colour.
The Custom tile shows a small tune icon so its
open-the-picker semantics is visually distinct from the
apply-immediately plugin tiles.
Editor bumped to 0.12.0 (panel toggle + LabVIEW-style port
colours).
Signed-off-by: flemming-it <stefan.a.flemming@googlemail.com>
This commit is contained in:
parent
c1c60d5434
commit
3db875c903
10 changed files with 105 additions and 31 deletions
|
|
@ -16,6 +16,7 @@ import 'package:shared_preferences/shared_preferences.dart';
|
|||
import '../data/hub.dart';
|
||||
import '../data/system_actions.dart';
|
||||
import '../l10n/app_localizations.dart';
|
||||
import '../main.dart' show StudioShellState;
|
||||
import '../theme/theme.dart';
|
||||
import '../theme/tokens.dart';
|
||||
import '../widgets/widgets.dart';
|
||||
|
|
@ -472,11 +473,36 @@ class _OnboardingChecklistState extends State<_OnboardingChecklist> {
|
|||
final theme = Theme.of(context);
|
||||
final l = AppLocalizations.of(context)!;
|
||||
final allDone = _aiOk && _mcpOk && _moduleOk && _flowOk;
|
||||
final items = <(bool, String, String)>[
|
||||
(_aiOk, l.welcomeChecklistAi, l.welcomeChecklistAiHint),
|
||||
(_mcpOk, l.welcomeChecklistMcp, l.welcomeChecklistMcpHint),
|
||||
(_moduleOk, l.welcomeChecklistModule, l.welcomeChecklistModuleHint),
|
||||
(_flowOk, l.welcomeChecklistFlow, l.welcomeChecklistFlowHint),
|
||||
// Each row gets an `onTap` that takes the operator to
|
||||
// the page where they can actually complete the item.
|
||||
// Replaces the old "read the hint then guess where the
|
||||
// feature lives" behaviour with a one-click hop.
|
||||
final shell = StudioShellState.of(context);
|
||||
final items = <(bool, String, String, VoidCallback?)>[
|
||||
(
|
||||
_aiOk,
|
||||
l.welcomeChecklistAi,
|
||||
l.welcomeChecklistAiHint,
|
||||
() => FaiSettingsDialog.show(context),
|
||||
),
|
||||
(
|
||||
_mcpOk,
|
||||
l.welcomeChecklistMcp,
|
||||
l.welcomeChecklistMcpHint,
|
||||
() => FaiSettingsDialog.show(context),
|
||||
),
|
||||
(
|
||||
_moduleOk,
|
||||
l.welcomeChecklistModule,
|
||||
l.welcomeChecklistModuleHint,
|
||||
shell == null ? null : () => shell.navigateTo('store'),
|
||||
),
|
||||
(
|
||||
_flowOk,
|
||||
l.welcomeChecklistFlow,
|
||||
l.welcomeChecklistFlowHint,
|
||||
shell == null ? null : () => shell.navigateTo('flows'),
|
||||
),
|
||||
];
|
||||
return Padding(
|
||||
padding: const EdgeInsets.only(bottom: FaiSpace.xxl),
|
||||
|
|
@ -533,6 +559,7 @@ class _OnboardingChecklistState extends State<_OnboardingChecklist> {
|
|||
done: items[i].$1,
|
||||
title: items[i].$2,
|
||||
hint: items[i].$3,
|
||||
onTap: items[i].$4,
|
||||
),
|
||||
],
|
||||
],
|
||||
|
|
@ -693,18 +720,23 @@ class _ChecklistRow extends StatelessWidget {
|
|||
final bool done;
|
||||
final String title;
|
||||
final String hint;
|
||||
/// Tap handler — takes the operator to the page where
|
||||
/// they can actually complete the item. null = no-op
|
||||
/// (e.g. AI item when System AI was already configured).
|
||||
final VoidCallback? onTap;
|
||||
|
||||
const _ChecklistRow({
|
||||
required this.done,
|
||||
required this.title,
|
||||
required this.hint,
|
||||
this.onTap,
|
||||
});
|
||||
|
||||
@override
|
||||
Widget build(BuildContext context) {
|
||||
final theme = Theme.of(context);
|
||||
final l = AppLocalizations.of(context)!;
|
||||
return Padding(
|
||||
final row = Padding(
|
||||
padding: const EdgeInsets.all(FaiSpace.lg),
|
||||
child: Row(
|
||||
crossAxisAlignment: CrossAxisAlignment.start,
|
||||
|
|
@ -747,6 +779,11 @@ class _ChecklistRow extends StatelessWidget {
|
|||
],
|
||||
),
|
||||
);
|
||||
// Wrap in InkWell only when a tap target exists so done
|
||||
// rows (no action needed) don't shimmer hover-feedback
|
||||
// they can't act on.
|
||||
if (onTap == null) return row;
|
||||
return InkWell(onTap: onTap, child: row);
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue