Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Input select #862

Merged
merged 13 commits into from
Oct 16, 2023
Merged

Input select #862

merged 13 commits into from
Oct 16, 2023

Conversation

allan-chagas-brisa
Copy link
Contributor

@allan-chagas-brisa allan-chagas-brisa commented Oct 6, 2023

Description

This PR implements the new component Input Select, which consists in a input next to a select button with a label that describes its usage.

Proposed Changes

  • User can pass the properties to the component via input following the interface IonInputSelectProps which has the following properties:
    @input() name: string; The property to link the input element with an possible label;
    @input() disabled = false; The property to disable the button and the input;
    @input() value = ''; The property that holds the value of the first input;
    @input() secondValue = ''; The property that holds the value of the second input;
    @input() selectOptions: SelectOption[] = defaultSelectOptions; The property where the user passes an array of options, of the type SelectOption, to be displayed on the dropdown from the select component;
    @output() valueChange = new EventEmitter(); The emmiter, that emmits an object of the type ValueToEmmit, every time the user inputs in the element;

How to Test

yarn test input-select

Screenshots

Gravacao.de.tela.de.06-10-2023.10.26.36.webm

View Storybook

Storybook

Compliance

  • I have verified that this change complies with our code and contribution policies.
  • I have verified that this change does not cause regressions and does not affect other parts of the code.

@allan-chagas-brisa allan-chagas-brisa self-assigned this Oct 6, 2023
@allan-chagas-brisa allan-chagas-brisa merged commit 0bd06a4 into main Oct 16, 2023
3 checks passed
@allan-chagas-brisa allan-chagas-brisa deleted the input-select branch October 16, 2023 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants