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

chore: remove the deprecated positionClass in tooltip #11067

Merged
merged 1 commit into from Mar 19, 2018
Merged

chore: remove the deprecated positionClass in tooltip #11067

merged 1 commit into from Mar 19, 2018

Conversation

DanielRuf
Copy link
Contributor

@DanielRuf DanielRuf commented Mar 15, 2018

positionClass was marked as deprecated a while ago.

I propose to plan this for 6.5 as this will also introduce a change to the code and the deprecated code will be removed.

Closes #11066

Copy link
Contributor

@ncoden ncoden left a comment

Choose a reason for hiding this comment

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

Like for #11038, positionClass was broke in v6.4.0 and never worked since. The position class is set on the template by Tooltip but only read on the element by Positionable. So LGTM 👍

@DanielRuf DanielRuf modified the milestones: 6.5.0, 6.4.4 Mar 16, 2018
@ncoden
Copy link
Contributor

ncoden commented Mar 16, 2018

And yes, good for v6.4.4 since we break nothing (already broken) but simply clean the code.

@ncoden
Copy link
Contributor

ncoden commented Mar 19, 2018

Note: v6.4.4 and v6.5.0 were moved to v6.5.0 and v6.6.0

@ncoden ncoden merged commit 5799bc9 into foundation:develop Mar 19, 2018
@DanielRuf DanielRuf deleted the chore/tooltip-remove-deprecated-position-class-11066 branch March 19, 2018 07:40
ncoden pushed a commit to ncoden/foundation-sites that referenced this pull request Jun 16, 2018
…-deprecated-position-class-11066 for v6.5.0

2f541a2 chore: remove the deprecated positionClass in tooltip

Signed-off-by: Nicolas Coden <nicolas@ncoden.fr>
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.

None yet

2 participants