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

Upgrade to bootstrap 3.0 #4

Closed
yoshuawuyts opened this issue Apr 13, 2013 · 29 comments
Closed

Upgrade to bootstrap 3.0 #4

yoshuawuyts opened this issue Apr 13, 2013 · 29 comments

Comments

@yoshuawuyts
Copy link
Contributor

Do you have any plans to upgrade to bootstrap 3.0 when it comes out? The rc has been out for a while, and I'm just curious if I can stick to your project.

@maxmx
Copy link
Owner

maxmx commented Apr 13, 2013

Yup, I'll translate it to stylus as soon as its officially out.

@yoshuawuyts
Copy link
Contributor Author

Sweet! Looking forward to it.

@raulghm
Copy link

raulghm commented May 25, 2013

Nice work, I also hope to see the upgrade to stylus :)

@abrkn
Copy link

abrkn commented May 26, 2013

👍

@skw
Copy link
Contributor

skw commented Jun 20, 2013

A heads up, v3 is moving from CSS reset to normalize.css. I've ported it here, I was porting v3 sometime ago but am moving towards styl and rework instead.

Also you will encounter some problems in media queries as stylus does not currently support variables as arguments in media queries (which there is a lot of in v3), there's offcourse a lot of really messy workarounds.

@maxmx
Copy link
Owner

maxmx commented Jun 21, 2013

Thanks for the heads up, do you have any pointers fr the workarounds for supporting the variables in the media queries?

@skw
Copy link
Contributor

skw commented Jun 22, 2013

Sort-off, there's a few workarounds that you can find by browsing issues (which I have attached to this comment) in the Stylus repo. The solution I had was creating a media mixin, however I wasn't really happy with it.

My solution

// media
media(args...)
  output = null
  for arg in args
    // check for tuple
    if arg[1]
      push(output,unquote('(%s: %s)' % (arg[0] arg[1])))
    else
      push(output,unquote(arg))

  unquote(s('%s',output))

Which can be used like this:

$_media = media(screen,'and',(min-width $screen-small))
@media $_media
  generate-large-grid-columns($grid-columns)

Besides the problem with @media queries Stylus is limited in that you can't write mixins as selectors, I had envisioned something like this:

// this does NOT work!
media(screen,'and',(min-width $screen-small))
  generate-large-grid-columns($grid-columns)

// neither does this:
@media media(screen,'and',(min-width $screen-small))
  generate-large-grid-columns($grid-columns)

Relevant Stylus issues:

Final Thoughts

These issues have been open for a while and the pull requests have largely been ignored. I can only guess that the reason is that TJHolowaychuk is too busy and is more interested in developing Styl and Rework.

Being somewhat of a hipster, I decided to abandon my stylus port of v3 to stylus and work on styl/rework projects (plugins and libraries).

@skw
Copy link
Contributor

skw commented Jul 29, 2013

BS 3 is out.

@abrkn
Copy link

abrkn commented Jul 29, 2013

Very interesting, will try soon

@abrkn
Copy link

abrkn commented Aug 1, 2013

Where are we on 3.0? Looking to upgrade.

@maxmx
Copy link
Owner

maxmx commented Aug 1, 2013

I'll be back from vacation on August 8th so in the following days.

On 2013-08-01, at 15:49, abrkn notifications@github.com wrote:

Where are we on 3.0? Looking to upgrade.


Reply to this email directly or view it on GitHub.

@maxmx
Copy link
Owner

maxmx commented Aug 14, 2013

I'll wait a bit for final release before making a translation seeing how the current release seems to be constantly updated.

@getvega
Copy link

getvega commented Aug 15, 2013

Thanks maxmx, can't wait to try it out.

@bumbu
Copy link
Contributor

bumbu commented Aug 20, 2013

Will new version make use of nib methods or it will use custom (mixins) functions?

@maxmx
Copy link
Owner

maxmx commented Aug 21, 2013

Allright release candidates are done, expect v3 in a few days.

@maxmx
Copy link
Owner

maxmx commented Aug 21, 2013

I'll try to make it nib-dependent

@getvega
Copy link

getvega commented Aug 21, 2013

Great news ! Thank you

@notslang
Copy link
Contributor

I'm working on a project that really needs BS3, but it uses stylus. Could you publish your progress with the port in a separate branch so I could pull from it (and maybe contribute)?

@skw
Copy link
Contributor

skw commented Aug 23, 2013

+1 for branch updates, if your lazy just add -A changes.

Btw I'm working on a less to stylus 'parser', it uses the less CSS parser if anyone's interested is using or contributing.

@maxmx
Copy link
Owner

maxmx commented Aug 23, 2013

Ssup dudes, I released it, I'm sure there are still some bugs left, but I ran the compiled css on the documentation site and everything looked good so far.

Also, I kept the mixins in instead of using nib, it was the easiest way to release this as fast as possible.

Please contribute if you find anything.

@liqweed
Copy link

liqweed commented Aug 25, 2013

@maxmx Could you please tag the release so that bower can resolve the new version?
I'm having a bad "No tag found that was able to satisfy 3.0.0" day :)

@maxmx
Copy link
Owner

maxmx commented Aug 25, 2013

@liqweed Done!

@liqweed
Copy link

liqweed commented Aug 25, 2013

Works!
Thanks 👍

@getvega
Copy link

getvega commented Aug 28, 2013

Everything fine on our end. Thanks !

I just had a max call stack error in the mixins

scale(ratio) {
  -webkit-transform: scale(ratio);
      -ms-transform: scale(ratio); // IE9+
          transform: scale(ratio);
}

Guess it'll be the same for rotate and other as well ?

@maxmx
Copy link
Owner

maxmx commented Aug 28, 2013

Right, its probably not used in bootstrap but you used it somewhere, I'll fix these soon.

@maxmx
Copy link
Owner

maxmx commented Aug 28, 2013

@liqweed I re-tagged the release with some bug fixes, I'd rather keep the 3.0.0 version so I deleted the tag and recreated it, you should update with bower if needed (especially if you were using any of the custom mixins as @getvega pointed out)

@liqweed
Copy link

liqweed commented Aug 29, 2013

@maxmx np, thanks for the heads up.

btw, I had a problem importing Bootstrap's theme.styl - would this by any chance address that?

@maxmx
Copy link
Owner

maxmx commented Aug 29, 2013

Yes i gave some love to the overlooked theme.styl file and even gave it a separate build using grunt.

On 2013-08-29, at 7:15, liqweed notifications@github.com wrote:

@maxmx np, thanks for the heads up.

btw, I had a problem importing Bootstrap's theme.styl - would this by any chance address that?


Reply to this email directly or view it on GitHub.

@arxpoetica
Copy link

@skw where can I find your less to stylus parser?

Awesome work everyone who built this. (@maxmx and party) +++1

@maxmx maxmx closed this as completed Sep 3, 2013
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

No branches or pull requests

10 participants