fix: Address Ingredients Parser UX overflow issues on mobile #4962
+17
−5
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What this PR does / why we need it:
Before
Desktop was ok
But mobile cropped error messages if there was more than one or they were long
![image](https://private-user-images.githubusercontent.com/365751/406810495-4ea4dab2-95db-4d0e-b049-246c996698d2.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mzg4NDE0NjMsIm5iZiI6MTczODg0MTE2MywicGF0aCI6Ii8zNjU3NTEvNDA2ODEwNDk1LTRlYTRkYWIyLTk1ZGItNGQwZS1iMDQ5LTI0NmM5OTY2OThkMi5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjUwMjA2JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI1MDIwNlQxMTI2MDNaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT04MDViNWNiMDU2MWI3ZDgxOGNjZTZlZTdkMzJmY2ZmODg0NGYzMDE3MjY4ZGNlYzQ2MTM1YmNkNTgzNDIxNzljJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.OsSYzi4yfcd6OQH2QnIOeYVZB7xv9zVeAn4FL_1KEho)
After:
![image](https://private-user-images.githubusercontent.com/365751/406810360-6c6829b6-3ecb-4633-b990-c728d5c83f36.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mzg4NDE0NjMsIm5iZiI6MTczODg0MTE2MywicGF0aCI6Ii8zNjU3NTEvNDA2ODEwMzYwLTZjNjgyOWI2LTNlY2ItNDYzMy1iOTkwLWM3MjhkNWM4M2YzNi5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjUwMjA2JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI1MDIwNlQxMTI2MDNaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT03OTZhMzgxMzE5ODc2ZDFhY2JlYmU5NWFlNDEyODMxNzgwZWQ2ZjJhMTMzZTNjODk5ZmUyZDMxOGFhMjg0OGJjJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.FvCCX2s68KdbyWRcyCt5WQVUMBBSUbeSTX49Eyi_fN4)
Which issue(s) this PR fixes:
#4954
Special notes for your reviewer:
This has a slight tradeoff in that the existing call to action ("Create") now relies on the user understanding what the + icon means, unless they hover over and read the title attribute.
The use of
flex-wrap
also looks a bit stupid, but the alternative was a list group; which looked bad on desktop.If the CSS framework has a btn-group-vertical on small screens class; I couldnt find it.
Testing
Tested locally, w/Chrome dev tools for mobile UX.