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

Vote button missing default styling in 3.5.1 #9966

Closed
wants to merge 5 commits into from
Closed

Vote button missing default styling in 3.5.1 #9966

wants to merge 5 commits into from

Conversation

N6REJ
Copy link
Contributor

@N6REJ N6REJ commented Apr 18, 2016

Pull Request for Issue # .

Summary of Changes

added btn-default to the "rate" button to provide default bootstrap styling in 3.5.1

Testing Instructions

create a article that has voting turned on.
inspect the "rate" button. You'll see that there is no default styling.
image

install the patch and you will see the styling added... and button will take on the default color style for
buttons.
image

@N6REJ N6REJ changed the title Votebutton Vote button missing default styling in 3.5.1 Apr 18, 2016
@andrepereiradasilva
Copy link
Contributor

i can't see any visual change in protostar after PR. But i can see the new css class.

@SharkyKZ
Copy link
Contributor

Is this meant for future Joomla versions in case a new Bootstrap version is bundled? Because .btn-default class doesn't exist in Bootstrap v2 currently included in Joomla.

@brianteeman
Copy link
Contributor

This PR is wrong as it is using a non existent class. That why @andrepereiradasilva couldn't see any change.

Check the bootstrap docs :)

@N6REJ
Copy link
Contributor Author

N6REJ commented Apr 19, 2016

sorry my bad.. I'm so used to bs3 never dawned on me bs2 would change the name of that single button class when all the rest are the same LOL...
it might be nice to leave it as bs3 prep but I understand if you want to close it brian. .sorry @andrepereiradasilva

@brianteeman
Copy link
Contributor

Sorry but closing this

@N6REJ N6REJ deleted the votebutton branch April 20, 2016 05:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants