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

Remove appearance from textarea #29455

Merged
merged 2 commits into from
Oct 2, 2019

Conversation

MartijnCuppens
Copy link
Member

This line was added to make inputs and textareas have the same border, but it seems it has a lot of unexpected side effects and support will be removed by the end of the year.

Closes #29430

@mdo
Copy link
Member

mdo commented Sep 27, 2019

No backport?

@MartijnCuppens
Copy link
Member Author

MartijnCuppens commented Sep 27, 2019

No, I added that in the latest reboot changes for v5

@XhmikosR
Copy link
Member

XhmikosR commented Sep 28, 2019

I contacted GH Support regarding the CI not firing up.

@XhmikosR
Copy link
Member

XhmikosR commented Oct 2, 2019

Never received a reply from support, but this should be fixed now anyway.

@MartijnCuppens MartijnCuppens force-pushed the master-mc-remove-appearance-from-textarea branch from 1091f04 to 5804e91 Compare October 2, 2019 14:35
@MartijnCuppens MartijnCuppens force-pushed the master-mc-remove-appearance-from-textarea branch from 5804e91 to 335364f Compare October 2, 2019 14:37
@MartijnCuppens
Copy link
Member Author

I've rebased the PR, tests are running now.

@XhmikosR
Copy link
Member

XhmikosR commented Oct 2, 2019

Yeah but there's another bug that the merge button is disabled even with approvals and checks passing.

I sent them an email, but at this point, I doubt I'll get a reply :P

@XhmikosR XhmikosR merged commit de8c651 into master Oct 2, 2019
@XhmikosR XhmikosR deleted the master-mc-remove-appearance-from-textarea branch October 2, 2019 19:37
ashfahan added a commit to fluid-framework/Fluid that referenced this pull request Oct 9, 2019
ashfahan added a commit to fluid-framework/Fluid that referenced this pull request Oct 13, 2019
Breaking changes:
🆙 💥 Use class `.Label` instead of label element to style controls like radio, switch, file and input floating labels for better web standards and web accessibility

Removed:
🔥 Remove padding from `lyt-sidebar` to let user have control over it properly
🔥 remove custom padding from `.menu`
🔥 Remove unnecessary `.btn.is-min` theme variant style

Fix:
🔨 Fix `.lyt-header.is-collapsed` not working
🔨 `.menu-items` not inheriting `font-size`
🔨 not inheriting color in `.menu` when using color utilities
🔨 Add padding in nested menu items to prevent text from going into overflow area

Added:
➕ 🚀 Add NEW dropdown menu component
➕ Add `border-hide` utility to hide border using transparent color
➕ Add `border-(direction)-none` utilities to remove border from specific direction
➕ Change header and sidebar color based on `$design-theme`
➕ Add active state of `btn.is-min`

Updated:
🆙 Set `text-align:start` on root to support RTL languages
🆙 🔀 Rearrange Theme colors to change there cascading priority
🆙 include all src and dist files in NPM package
🆙 Align icon in center when parent has display:flex
🆙 Flexible header height and flex wrap
🆙 Update From bootstrap reboot (twbs/bootstrap#29455)
🆙 override styles instead of increasing specificity, to make style overridable
🆙 `lyt-sidebar` are static by default, they can be made fixed using using `fixed` utility class.
🆙 Lighten .menu-item Background color
🆙 inherit `text-align` value in menu-item rather than explicit overriding
🆙 Use `display:block` on menu-items instead of flex to support `text-align` utility
🆙 Apply animation on menu items only if they were with `.menu-trigger`
🆙 Apply nested menu styles on direct child this will allow menu style to stop conflicting inside other components and universal selector `*` to be limited to direct children
🆙 Decrease nested menu animation duration
🆙 Increase before border-color transition duration in `.menu-item` to shorten and decrease chances of user specifies value more than transition duration
🆙 use scale to flip caret in menu instead of rotate

Design:
💅 Don't override text `color` in `menu-item`

Dev:
➕ :octocat: Add `fill-direction` parameter for @mixin animation
🔖 :octocat: Add variables `$menu-trigger-caret-color` and `$menu-divider-padding`
🔖 :octocat: Add variables for nested Menu animation duration `$$nestedMenu-animationduration`
🔖 :octocat: rename variable `$nestedMenu-marginleft` to `$nestedMenu-indent`
🔖 :octocat: rename `$button-transition-minimal` to `$button-transition-minimal-before` to be more specific in naming convention
🔖 🔨 :octocat: Fix variable name from bg to background
🔖 🔀 :octocat: Arrange and reorder setting variables in menu
➕ ✅ Test for invalid @function `clr-step` values
🆙 :octocat: Make Triangle shape size equal to the size specified preventing extra style to used to align middle
📦 :octocat: Update package `find-unused-sass-variables`
lucanos pushed a commit to lucanos/bootstrap that referenced this pull request Oct 27, 2019
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.

Reboot: remove textarea -webkit-appearance: textfield
3 participants