Skip to content
This repository was archived by the owner on Oct 8, 2021. It is now read-only.

Deprecate style options #7348

Closed

Conversation

allsteady
Copy link
Contributor

Hi all

I add css rules for negating a style option.

  • data-corner
  • data-shadow

button-negate

If there are any problems or mistakes on that PR, please let me know. Thanks in advance.

Fixes gh-7303
Ref #7304

@gabrielschulhof
Copy link

@allsteady That's really good so far. Please also modify the elements which use data-corners="false" to use class=... ui-corner-none ... or data-wrapper-class=... ui-corner-none .... For example, on https://github.com/jquery/jquery-mobile/blob/39fe09ebf3d40321b7d7c389c1b5676b01093971/demos/button/index.php#L57 please replace data-corners="none" with data-wrapper-class="ui-corner-none".

@gabrielschulhof
Copy link

@allsteady please do the same for data-shadow="false".

/* Negate Class
-----------------------------------------------------------------------------------------------------------*/

.ui-corenr-none, .ui-corner-none.ui-corner-all {

Choose a reason for hiding this comment

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

There's a typo here. It should be .ui-corner-none

@jaspermdegroot
Copy link
Contributor

@allsteady

Please note that there should not be a period (.) at the end of the first line of the commit message (see http://contribute.jquery.org/commits-and-pull-requests/#commit-guidelines).
For commits that add/change a demo for a specific widget we use the name of the widget as component in the commit message instead of "Demos:".
For example: "Button: Added negate style options demo"
We will change the message of the existing commits, so it's not a problem. I just wanted to make you aware of it.
Thanks!

@gabrielschulhof
Copy link

@allsteady Could you please also use your full name as the commit author?

@jaspermdegroot jaspermdegroot changed the title Button: Remove some style options form the widget. Button: Deprecate style options May 27, 2014
@jaspermdegroot jaspermdegroot changed the title Button: Deprecate style options Deprecate style options May 27, 2014
@@ -28,4 +28,4 @@
width: 1px;
overflow: hidden;
clip: rect(1px,1px,1px,1px);
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Missing blank line at EOF

@arschmitz
Copy link
Contributor

@allsteady do you think you could add a comment inthe button widget above the options that are deprecated saying that they are deprecated in 1.5 and will be removed in 1.6

@gabrielschulhof
Copy link

@arschmitz @allsteady I would hold off on saying they will be removed in 1.6, because we may not finish all style option removal, and IINM we've agreed that, if it takes longer, we keep those for which we've created CSS-based replacements deprecated until we remove them all at the same time in a future version (1.7.0 if we don't finish creating replacements in time for 1.5.0).

@gabrielschulhof
Copy link

@allsteady you should nevertheless add a comment saying that they are deprecated in 1.5.0. That part is for sure.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) when pulling abeccbc on allsteady:remove-style-option-in-button into 00fb4e2 on jquery:master.

@jaspermdegroot
Copy link
Contributor

We have to change the component in the commit messages:
592ae79: Button --> Theme
abeccbc: Demos --> Button

@gabrielschulhof
Copy link

@allsteady will submit a second PR against the newly created branch "style-option-deprecation".

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling cb26f68 on allsteady:remove-style-option-in-button into 44f503d on jquery:master.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Button: Deprecate style options
5 participants