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

Fix SelectBox width of items elements #330

Merged
merged 2 commits into from Jan 21, 2019
Merged

Conversation

migueliriano
Copy link
Contributor

@migueliriano migueliriano commented Jan 17, 2019

Release Type: Non-Breaking Feature

Fixes https://x-team-internal.atlassian.net/browse/XP-2721

Description

Set up a max-width for SelectBox items and have different words length without horizontal scroll.

Checklist

Before submitting a pull request, please make sure the following is done:

  • set yourself as an assignee
  • set appropriate labels for a PR (In Review or In Progress depending on its status)
  • move respective JIRA issue to the IN REVIEW column
  • make sure your code lints (npm run lint)
  • Flow typechecks passed (npm run flow)
  • Snapshots tests passed (npm run jest)
  • check cleanup tasks (https://github.com/x-team/auto/labels/cleanup) and take a suitable small one (if exists) in a related area of the current changes
  • component's documentation (.stories.js file) is changed or added accordingly to reflect any new or updated use cases or variants usage.
  • if any snapshots have been changed, verify that component still works and looks as expected and update the changed snapshot
  • manually tested the component by running it in the browser and checked nothing is broken and operates as expected!

Related PRs

branch PR
XP-2721-fix-dropdown-width Auto

Impacted Areas in Application

  • SelectBox component

Screenshots

dropdown_with_long_word

Copy link
Contributor

@galileo galileo left a comment

Choose a reason for hiding this comment

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

Looks good!

Copy link
Collaborator

@nicksp nicksp left a comment

Choose a reason for hiding this comment

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

Good work! Can confirm this now renders correctly.

@nicksp nicksp added Done and removed In Review labels Jan 21, 2019
@nicksp nicksp merged commit cf3183c into release-18 Jan 21, 2019
@nicksp nicksp deleted the XP-2721-selectbox-fix-width branch January 21, 2019 11:41
nicksp pushed a commit that referenced this pull request Jan 21, 2019
* Fix SelectBox width of items elements

* Build app

# Conflicts:
#	lib/auto-ui.js.map
@nicksp nicksp mentioned this pull request Jan 21, 2019
nicksp added a commit that referenced this pull request Jan 21, 2019
* Truncate long attached file names (#320)

* Truncate long attached file names

* Update lineHeight

* Build app

# Conflicts:
#	lib/auto-ui.js.map

*  Implement RichTextEditor component in Note component  (#321)

* Implement RichTextEditor component in Note component

+ Add `disabled` prop to `RichTextEditor`

Closes XP-2673

* Build lib + update snapshots

* Rename `markdown` prop to `value`

# Conflicts:
#	lib/auto-ui.js.map

* Fix SelectBox's dissmissing messages behaviour (#319)

* Fix SelectBox's dissmissing messages behaviour

* Include dissmising of deletion messages at SelectBox unmounting

* Prevent unnecessary action triggers

# Conflicts:
#	lib/auto-ui.js.map

* Add default toolbar items (#326)

* Add default toolbar items

* Remove export elements and update stories book

* Rollback css changes

* Declare only one default toolbar variable

* Build app

# Conflicts:
#	lib/auto-ui.js.map

* Update top profile design (#325)

* Upgrade button and SvgIcon component to adapt new style on auto (#322)

* Upgrade button and Svg component to adapt new style on auto

* Add new props to button component: icon, iconProps and contentStyle

* Add new color to Svgicon and new custom style for button components

* Build app

*  Add Activity Header (#324)

* Add time SvgIcon

* Add Activity Header

* Fix flexbox for grid

* Add build

* Address PR requested changes

* Fix activity description `line-height` + remove `min-width`

* Remove `max-width`

* Use `rem` instead of `px` fot font-size + Fix snapshots

* Update padding button fro sourceSansPro content style

* Build app

# Conflicts:
#	lib/auto-ui.js.map

*  Add bottom padding to RichTextEditor (#329)

* Add bottom padding for rich text editor

* Update test

* Add build

* Fix SelectBox width of items elements (#330)

* Fix SelectBox width of items elements

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

Successfully merging this pull request may close these issues.

None yet

4 participants