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

Use jquery.sceditor.mybb.css from plugin sceditor for Mybb. And fix too #667 #666 #662 #663 #420 #668

Closed
wants to merge 4 commits into from

Conversation

martec
Copy link
Contributor

@martec martec commented Jun 1, 2014

jquery.sceditor.mybb.css from https://github.com/samclarke/SCEditor-MyBB/tree/master/jscripts/sceditor

Better use jquery.sceditor.mybb.css instead jquery.sceditor.default.min.css has better style for blockquote, blockcode etc..

jquery.sceditor.mybb.css
http://i.imgur.com/V7u7TiE.png

jquery.sceditor.default.min.css
http://i.imgur.com/hZUOtKZ.png

And i don´t recommend use minify file in this case, because this file is used too to put style for pre tag (used in spoiler tag) etc. Simply put this file is often used for customization.

This pull fix too #667 #666 #662 #663 #420 . So case to apply this, does not apply to others.

@martec martec changed the title Use jquery.sceditor.mybb.css from plugin sceditor for Mybb Use jquery.sceditor.mybb.css from plugin sceditor for Mybb. And fix too #667 #666 #662 #663 Jun 1, 2014
@martec martec changed the title Use jquery.sceditor.mybb.css from plugin sceditor for Mybb. And fix too #667 #666 #662 #663 Use jquery.sceditor.mybb.css from plugin sceditor for Mybb. And fix too #667 #666 #662 #663 #420 Jun 1, 2014
@DiogoParrinha
Copy link
Contributor

@martec we only accept one PR per issue otherwise if it doesn't fix A but fixes B and C we cannot accept it.

@martec
Copy link
Contributor Author

martec commented Jun 1, 2014

@PirataNervo
fix other problem is only consequence when change to jquery.sceditor.mybb.css

@DiogoParrinha
Copy link
Contributor

I'll manually fix this issue if you don't submit another PR in the future - I appreciate the fix!

@martec
Copy link
Contributor Author

martec commented Jun 1, 2014

@PirataNervo
ok... thanks...

@martec
Copy link
Contributor Author

martec commented Jun 1, 2014

close this because i pull request another that solve problem with commits

@martec martec closed this Jun 1, 2014
@martec martec deleted the scmybbcss branch June 1, 2014 12:21
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

2 participants