You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by pa...@apache.org on 2022/02/07 21:24:01 UTC
[beam] branch master updated: [BEAM-13820] Changed color of delete icon in pipeline options dropdown, removed unused imports
This is an automated email from the ASF dual-hosted git repository.
pabloem pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/beam.git
The following commit(s) were added to refs/heads/master by this push:
new de7e9cd [BEAM-13820] Changed color of delete icon in pipeline options dropdown, removed unused imports
new 63d9577 Merge pull request #16758 from [BEAM-13820] [Playground] Outline delete icon in Pipeline options
de7e9cd is described below
commit de7e9cdfbdc39dac7a204c378cec5d6ec3bb462a
Author: Alexander <al...@akvelon.com>
AuthorDate: Mon Feb 7 17:12:34 2022 +0400
[BEAM-13820] Changed color of delete icon in pipeline options dropdown, removed unused imports
---
.../pipeline_options_dropdown/pipeline_options_dropdown_body.dart | 2 --
.../components/pipeline_options_dropdown/pipeline_options_form.dart | 6 +++++-
2 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/playground/frontend/lib/modules/editor/components/pipeline_options_dropdown/pipeline_options_dropdown_body.dart b/playground/frontend/lib/modules/editor/components/pipeline_options_dropdown/pipeline_options_dropdown_body.dart
index f2ae442..aa6532a 100644
--- a/playground/frontend/lib/modules/editor/components/pipeline_options_dropdown/pipeline_options_dropdown_body.dart
+++ b/playground/frontend/lib/modules/editor/components/pipeline_options_dropdown/pipeline_options_dropdown_body.dart
@@ -18,7 +18,6 @@
import 'package:flutter/material.dart';
import 'package:flutter_gen/gen_l10n/app_localizations.dart';
-import 'package:playground/config/theme.dart';
import 'package:playground/constants/colors.dart';
import 'package:playground/constants/sizes.dart';
import 'package:playground/modules/editor/components/pipeline_options_dropdown/pipeline_option_model.dart';
@@ -26,7 +25,6 @@ import 'package:playground/modules/editor/components/pipeline_options_dropdown/p
import 'package:playground/modules/editor/components/pipeline_options_dropdown/pipeline_options_dropdown_separator.dart';
import 'package:playground/modules/editor/components/pipeline_options_dropdown/pipeline_options_form.dart';
import 'package:playground/modules/editor/parsers/run_options_parser.dart';
-import 'package:playground/modules/notifications/components/notification.dart';
const kOptionsTabIndex = 0;
const kRawTabIndex = 1;
diff --git a/playground/frontend/lib/modules/editor/components/pipeline_options_dropdown/pipeline_options_form.dart b/playground/frontend/lib/modules/editor/components/pipeline_options_dropdown/pipeline_options_form.dart
index 1d2179f..868ee4b 100644
--- a/playground/frontend/lib/modules/editor/components/pipeline_options_dropdown/pipeline_options_form.dart
+++ b/playground/frontend/lib/modules/editor/components/pipeline_options_dropdown/pipeline_options_form.dart
@@ -20,6 +20,7 @@ import 'package:flutter/material.dart';
import 'package:flutter_gen/gen_l10n/app_localizations.dart';
import 'package:collection/collection.dart';
import 'package:playground/config/theme.dart';
+import 'package:playground/constants/colors.dart';
import 'package:playground/constants/sizes.dart';
import 'package:playground/modules/editor/components/pipeline_options_dropdown/pipeline_option_label.dart';
import 'package:playground/modules/editor/components/pipeline_options_dropdown/pipeline_option_model.dart';
@@ -72,7 +73,10 @@ class PipelineOptionsForm extends StatelessWidget {
child: IconButton(
iconSize: kIconSizeMd,
splashRadius: kIconButtonSplashRadius,
- icon: const Icon(Icons.delete_outlined),
+ icon: const Icon(
+ Icons.delete_outlined,
+ color: kLightPrimary,
+ ),
color: ThemeColors.of(context).grey1Color,
onPressed: () => onDelete(index),
),