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

♻️ Refactor Search to use new InputWrapper #2481

Merged
merged 9 commits into from
Sep 13, 2022

Conversation

mimarz
Copy link
Contributor

@mimarz mimarz commented Sep 12, 2022

resolves #2473

@mimarz mimarz changed the title ♻️ RefactorSearch to use new Input ♻️ RefactorSearch to use new InputWrapper Sep 13, 2022
@mimarz mimarz changed the base branch from develop to feat/2336-inputwrapper September 13, 2022 08:24
@mimarz mimarz marked this pull request as ready for review September 13, 2022 08:24
@mimarz mimarz changed the title ♻️ RefactorSearch to use new InputWrapper ♻️ Refactor Search to use new InputWrapper Sep 13, 2022
Copy link
Collaborator

@oddvernes oddvernes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@mimarz mimarz merged commit 8e23095 into feat/2336-inputwrapper Sep 13, 2022
@mimarz mimarz deleted the feat/MIM-2473-search-inputwrapper branch September 13, 2022 13:05
mimarz added a commit that referenced this pull request Sep 21, 2022
* ♻️ Initial rewrite of Search

* ♻️ Re-introduce height on input due to compact

* 📝 Updated search docs

* ♻️ Introduces Inputwrapper

* ✅ Updated test

* 🐛 Fixed bugs

* 🗑️ Clean code

* Updated snapshots

* 🔥 Removed unused files
mimarz added a commit that referenced this pull request Oct 4, 2022
* ♻️ Initial rewrite of Search

* ♻️ Re-introduce height on input due to compact

* 📝 Updated search docs

* ♻️ Introduces Inputwrapper

* ✅ Updated test

* 🐛 Fixed bugs

* 🗑️ Clean code

* Updated snapshots

* 🔥 Removed unused files
mimarz added a commit that referenced this pull request Oct 7, 2022
* ♻️ Initial rewrite of Search

* ♻️ Re-introduce height on input due to compact

* 📝 Updated search docs

* ♻️ Introduces Inputwrapper

* ✅ Updated test

* 🐛 Fixed bugs

* 🗑️ Clean code

* Updated snapshots

* 🔥 Removed unused files
mimarz added a commit that referenced this pull request Oct 11, 2022
* ♻️ Initial rewrite of Search

* ♻️ Re-introduce height on input due to compact

* 📝 Updated search docs

* ♻️ Introduces Inputwrapper

* ✅ Updated test

* 🐛 Fixed bugs

* 🗑️ Clean code

* Updated snapshots

* 🔥 Removed unused files
mimarz added a commit that referenced this pull request Oct 13, 2022
* ♻️ Initial rewrite of Search

* ♻️ Re-introduce height on input due to compact

* 📝 Updated search docs

* ♻️ Introduces Inputwrapper

* ✅ Updated test

* 🐛 Fixed bugs

* 🗑️ Clean code

* Updated snapshots

* 🔥 Removed unused files
mimarz added a commit that referenced this pull request Oct 18, 2022
* ♻️ Initial rewrite of Search

* ♻️ Re-introduce height on input due to compact

* 📝 Updated search docs

* ♻️ Introduces Inputwrapper

* ✅ Updated test

* 🐛 Fixed bugs

* 🗑️ Clean code

* Updated snapshots

* 🔥 Removed unused files
mimarz added a commit that referenced this pull request Oct 18, 2022
* ✨ Adornments support in `Input` component (#2354)

* 🚧 WIP

* 📝 Added story

* wip

* 🚧 WIP

* 🚧 WIP

* 🚧 wip

* 🚚 Moved over `Input` adornment changes

* ♻️ Added `OldInput` for interim development

* 🐛 Fixed textarea outline

* ♻️ back to new input

* 🔥 removed OldInput

* Testing something old

* updated snapshots

* its something

* ♻️ working version before testing styles

* ♻️ Further testing and tweaking

* 📸 ✅ Updated test & snapshots

* ♻️ working example

* 🚚 Moved typographymixin

* ♻️ Started cleaning up helpertext

* 📸 Updated snapshots

* Working example with colored helpertext

* 🚧 WIP

* 🚧 WIP

* tweaks

* Added description

* ♻️ Textfield using `InputWrapper` (#2416)

* ✨ Adornments support in `Input` component (#2354)

* 🚧 WIP

* 📝 Added story

* wip

* 🚧 WIP

* 🚧 WIP

* 🚧 wip

* 🚚 Moved over `Input` adornment changes

* ♻️ Added `OldInput` for interim development

* 🐛 Fixed textarea outline

* ♻️ back to new input

* 🔥 removed OldInput

* Testing something old

* updated snapshots

* its something

* ♻️ working version before testing styles

* ♻️ Further testing and tweaking

* 📸 ✅ Updated test & snapshots

* ♻️ working example

* 🚚 Moved typographymixin

* ♻️ Started cleaning up helpertext

* ♻️ Working with just input

* 🚧 WIP

* ♻️ Fixed helper text & colors

* ♻️ changed to usecallback

* ⚰️ Removed unused files

* ♻️ improvements to input

* ♻️ Simplified adornment width

* 🐛 Fixed new variant in Textarea

* ♻️ Textfield with discriminating union

* ♻️ re-introduced old proxy component

* Fiex typing

* 🗑️ Cleaning up files

* Added overridablecomponent to input

* 🏷️ Made as optional in overridableComponent

* 🐛 Fix aria-describedby when no helpertext is provided

* ♻️ Updated Textarea with input

* Cleaning up helpertext

* ♻️ Tweak adornments

* 📸 Update snapshots

* ♻️ Working textarea within Textfield

* ⏪ Reverted adornment styling changes

* Cleanup

* ♻️ Update Textfield rstories

* 📸 Update snapshots

* ♻️ Re-added support for manual width

* ♻️ Refactor `Search` to use new `InputWrapper` (#2481)

* ♻️ Initial rewrite of Search

* ♻️ Re-introduce height on input due to compact

* 📝 Updated search docs

* ♻️ Introduces Inputwrapper

* ✅ Updated test

* 🐛 Fixed bugs

* 🗑️ Clean code

* Updated snapshots

* 🔥 Removed unused files

* 🐛 Fixed height for when multilines

* 📸 Updated snapshots

* 🚨 Cleaned up some linting warnings

* 📝 Wording
denektenina pushed a commit to denektenina/design-system that referenced this pull request Oct 19, 2022
* ✨ Adornments support in `Input` component (equinor#2354)

* 🚧 WIP

* 📝 Added story

* wip

* 🚧 WIP

* 🚧 WIP

* 🚧 wip

* 🚚 Moved over `Input` adornment changes

* ♻️ Added `OldInput` for interim development

* 🐛 Fixed textarea outline

* ♻️ back to new input

* 🔥 removed OldInput

* Testing something old

* updated snapshots

* its something

* ♻️ working version before testing styles

* ♻️ Further testing and tweaking

* 📸 ✅ Updated test & snapshots

* ♻️ working example

* 🚚 Moved typographymixin

* ♻️ Started cleaning up helpertext

* 📸 Updated snapshots

* Working example with colored helpertext

* 🚧 WIP

* 🚧 WIP

* tweaks

* Added description

* ♻️ Textfield using `InputWrapper` (equinor#2416)

* ✨ Adornments support in `Input` component (equinor#2354)

* 🚧 WIP

* 📝 Added story

* wip

* 🚧 WIP

* 🚧 WIP

* 🚧 wip

* 🚚 Moved over `Input` adornment changes

* ♻️ Added `OldInput` for interim development

* 🐛 Fixed textarea outline

* ♻️ back to new input

* 🔥 removed OldInput

* Testing something old

* updated snapshots

* its something

* ♻️ working version before testing styles

* ♻️ Further testing and tweaking

* 📸 ✅ Updated test & snapshots

* ♻️ working example

* 🚚 Moved typographymixin

* ♻️ Started cleaning up helpertext

* ♻️ Working with just input

* 🚧 WIP

* ♻️ Fixed helper text & colors

* ♻️ changed to usecallback

* ⚰️ Removed unused files

* ♻️ improvements to input

* ♻️ Simplified adornment width

* 🐛 Fixed new variant in Textarea

* ♻️ Textfield with discriminating union

* ♻️ re-introduced old proxy component

* Fiex typing

* 🗑️ Cleaning up files

* Added overridablecomponent to input

* 🏷️ Made as optional in overridableComponent

* 🐛 Fix aria-describedby when no helpertext is provided

* ♻️ Updated Textarea with input

* Cleaning up helpertext

* ♻️ Tweak adornments

* 📸 Update snapshots

* ♻️ Working textarea within Textfield

* ⏪ Reverted adornment styling changes

* Cleanup

* ♻️ Update Textfield rstories

* 📸 Update snapshots

* ♻️ Re-added support for manual width

* ♻️ Refactor `Search` to use new `InputWrapper` (equinor#2481)

* ♻️ Initial rewrite of Search

* ♻️ Re-introduce height on input due to compact

* 📝 Updated search docs

* ♻️ Introduces Inputwrapper

* ✅ Updated test

* 🐛 Fixed bugs

* 🗑️ Clean code

* Updated snapshots

* 🔥 Removed unused files

* 🐛 Fixed height for when multilines

* 📸 Updated snapshots

* 🚨 Cleaned up some linting warnings

* 📝 Wording
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update Search to use InputWrapper
2 participants