Skip to content

Commit 1c84be7

Browse files
committed
Clean up
1 parent 8b5b6bf commit 1c84be7

1 file changed

Lines changed: 22 additions & 17 deletions

File tree

packages/devtools_app/lib/src/standalone_ui/ide_shared/property_editor/property_editor_panel.dart

Lines changed: 22 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -82,8 +82,6 @@ class _PropertyEditorPanelState extends State<PropertyEditorPanel> {
8282
class _PropertyEditorConnectedPanel extends StatefulWidget {
8383
const _PropertyEditorConnectedPanel(this.editor, {required this.controller});
8484

85-
static const footerHeight = 25.0;
86-
8785
final EditorClient editor;
8886
final PropertyEditorController controller;
8987

@@ -155,6 +153,8 @@ class _PropertyEditorConnectedPanelState
155153
class _PropertyEditorFooter extends StatelessWidget {
156154
const _PropertyEditorFooter();
157155

156+
static const _footerHeight = 25.0;
157+
158158
@override
159159
Widget build(BuildContext context) {
160160
final theme = Theme.of(context);
@@ -165,14 +165,20 @@ class _PropertyEditorFooter extends StatelessWidget {
165165
color: colorScheme.surface,
166166
border: Border(top: BorderSide(color: Theme.of(context).focusColor)),
167167
),
168-
height: _PropertyEditorConnectedPanel.footerHeight,
168+
height: _footerHeight,
169169
padding: const EdgeInsets.symmetric(vertical: densePadding),
170-
alignment: Alignment.center,
171170
child: Row(
171+
crossAxisAlignment: CrossAxisAlignment.end,
172172
children: [
173-
const Spacer(),
174173
if (documentationLink != null)
175-
_DocsLink(documentationLink: documentationLink),
174+
Padding(
175+
padding: const EdgeInsets.only(left: denseSpacing),
176+
child: _DocsLink(
177+
documentationLink: documentationLink,
178+
color: colorScheme.onSurface,
179+
),
180+
),
181+
const Spacer(),
176182
ReportFeedbackButton(color: colorScheme.onSurface),
177183
],
178184
),
@@ -190,23 +196,22 @@ class _PropertyEditorFooter extends StatelessWidget {
190196
}
191197

192198
class _DocsLink extends StatelessWidget {
193-
const _DocsLink({required this.documentationLink});
199+
const _DocsLink({required this.documentationLink, required this.color});
194200

201+
final Color color;
195202
final String documentationLink;
196203

197204
@override
198205
Widget build(BuildContext context) {
199-
return RichText(
200-
text: GaLinkTextSpan(
201-
link: GaLink(
202-
display: 'documentation',
203-
url: documentationLink,
204-
gaScreenName: gac.PropertyEditorSidebar.id,
205-
gaSelectedItemDescription:
206-
gac.PropertyEditorSidebar.documentationLink,
207-
),
208-
context: context,
206+
return LinkIconLabel(
207+
icon: Icons.library_books_outlined,
208+
link: GaLink(
209+
display: 'Docs',
210+
url: documentationLink,
211+
gaScreenName: gac.PropertyEditorSidebar.id,
212+
gaSelectedItemDescription: gac.PropertyEditorSidebar.documentationLink,
209213
),
214+
color: color,
210215
);
211216
}
212217
}

0 commit comments

Comments
 (0)