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 dismissible spelling #13845

Merged
merged 1 commit into from
Jun 17, 2014
Merged

Fix dismissible spelling #13845

merged 1 commit into from
Jun 17, 2014

Conversation

BBosman
Copy link
Contributor

@BBosman BBosman commented Jun 17, 2014

Based on #10404.

In the code we use dismissable, while it should be dismissible. (Dictionary entry)

I changed all references in the docs, but kept a copy of the old class name in the alerts.less file, marked as deprecated, for backwards compatibility.

@cvrebert cvrebert added the css label Jun 17, 2014
@cvrebert cvrebert added this to the v3.2.1 milestone Jun 17, 2014
@mdo
Copy link
Member

mdo commented Jun 17, 2014

Getting a merge conflict—perhaps drop the compiled CSS?

@mdo mdo modified the milestones: v3.2.1, v3.2.0 Jun 17, 2014
@mdo
Copy link
Member

mdo commented Jun 17, 2014

Btw, I think you can't see that there is a merge conflict here—I'm working on that at GitHub, too. Expect a change for that in the near future (and apologies for the shitty experience the last few months on that). <3

@BBosman
Copy link
Contributor Author

BBosman commented Jun 17, 2014

I dropped the compiled CSS as requested. Hope that fixes the conflict.

@XhmikosR
Copy link
Member

Nope, still conflicts :/

@BBosman
Copy link
Contributor Author

BBosman commented Jun 17, 2014

What file(s) are having the conflict?

@XhmikosR
Copy link
Member

I don't know either, the interface doesn't show... Anyway, just fetch the upstream master and rebase; it should be ok then.

@XhmikosR
Copy link
Member

...but my guess would be it's because of 5b9e2bf so popovers.html.

XhmikosR added a commit that referenced this pull request Jun 17, 2014
@XhmikosR XhmikosR merged commit 316e0c7 into twbs:master Jun 17, 2014
@BBosman BBosman deleted the dismissible branch June 17, 2014 22:03
RobJacobs added a commit to angular-ui/bootstrap that referenced this pull request Jun 4, 2015
The use of alert-dismissable is now deprecated in correspondence with twbs/bootstrap#13845
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.

4 participants