Skip to content
This repository has been archived by the owner on May 20, 2023. It is now read-only.

material-yes-button-text-color mixin don't work with stepper component #279

Closed
muzuro opened this issue Jun 28, 2018 · 2 comments
Closed

Comments

@muzuro
Copy link

muzuro commented Jun 28, 2018

I created component with template:

<div class="teal-themed">
    <material-stepper legalJumps="backwards"
                      orientation="horizontal"
                      size="default">
        <template step name="Consent">
            <div>
            </div>
        </template>
        <template step name="Strips">
            <div>
            </div>
        </template>
    </material-stepper>
</div>

and scss:

@import 'package:angular_components/css/material/material';
@import 'package:angular_components/material_stepper/mixins';
@import 'package:angular_components/material_yes_no_buttons/mixins';

:host {
  @include material-stepper-theme (
          $selector: '.teal-themed',
          $step-color: $mat-teal-500,
          $button-color: $mat-blue-400
  );
  .teal-themed {
    @include material-no-button-text-color($mat-teal-500);
    @include material-yes-button-text-color($mat-teal-500);
  }
}

Strange that this code override color for no button but is ignored for yes. Styles from asset:angular_components/lib/material_yes_no_buttons/material_yes_no_buttons.dart is not override by mixin - as I see it in chrome dev tools.

@TedSander
Copy link
Contributor

Thanks for the bug. The yes button is raised and highlighted which is causing it to not have a great enough specificity on the CSS selector. I'll try to get a fix in the next sync.

nshahan pushed a commit that referenced this issue Jul 17, 2018
…utton.

See github issue: #279

PiperOrigin-RevId: 203003871
nshahan pushed a commit that referenced this issue Jul 18, 2018
…utton.

See github issue: #279

PiperOrigin-RevId: 203003871
@nshahan
Copy link
Contributor

nshahan commented Jul 20, 2018

Should be resolved in v0.9.0-beta+1. Feel free to reopen if needed.

@nshahan nshahan closed this as completed Jul 20, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants