Skip to content

Commit

Permalink
Merge pull request apache#16758 from [BEAM-13820] [Playground] Outlin…
Browse files Browse the repository at this point in the history
…e delete icon in Pipeline options

[BEAM-13820] [Playground] Outline delete icon in Pipeline options
  • Loading branch information
pabloem authored Feb 7, 2022
2 parents 5e8c258 + de7e9cd commit 63d9577
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -18,15 +18,13 @@

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';
import 'package:playground/modules/editor/components/pipeline_options_dropdown/pipeline_options_dropdown_input.dart';
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;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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';
Expand Down Expand Up @@ -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),
),
Expand Down

0 comments on commit 63d9577

Please sign in to comment.