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

UIIN-2493 Make Inventory search and browse query boxes expandable #2322

Merged
merged 4 commits into from
Oct 30, 2023

Conversation

BogdanDenis
Copy link
Contributor

@BogdanDenis BogdanDenis commented Oct 26, 2023

Description

Make Inventory search and browse query boxes expandable

Screenshots

chrome_aBXCGEl7SF.mp4

Issues

UIIN-2493

Related PRs

folio-org/stripes-smart-components#1407

@github-actions
Copy link

github-actions bot commented Oct 26, 2023

Jest Unit Test Statistics

    1 files  ±0  226 suites  ±0   12m 2s ⏱️ +6s
888 tests ±0  886 ✔️ ±0  2 💤 ±0  0 ±0 
893 runs  ±0  891 ✔️ ±0  2 💤 ±0  0 ±0 

Results for commit ab8a210. ± Comparison against base commit e583600.

♻️ This comment has been updated with latest results.

@@ -1178,6 +1178,7 @@ class InstancesList extends React.Component {
searchableIndexesPlaceholder={null}
initialResultCount={INITIAL_RESULT_COUNT}
initiallySelectedRecord={getItem(`${namespace}.${segment}.lastOpenRecord`)}
inputType="textarea"
Copy link
Contributor

Choose a reason for hiding this comment

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

Textarea looks much higher to me
image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed
image

@@ -3,6 +3,7 @@

## [10.1.0] IN PROGRESS

* Make Inventory search and browse query boxes expandable. Refs UIIN-2493.
Copy link
Contributor

Choose a reason for hiding this comment

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

Should the Browse query box be expandable?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@BogdanDenis BogdanDenis merged commit 9460752 into master Oct 30, 2023
4 checks passed
@BogdanDenis BogdanDenis deleted the UIIN-2493 branch October 30, 2023 11:51
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.

4 participants