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

resolves #19 - migrate to Foundation 5 #23

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mojavelinux
Copy link
Member

  • migrate from Foundation 4 to Foundation 5
  • upgrade to Compass 1.0.1
  • make stylesheet changes necessary to accomodate upgrades
  • add Bower configuration
  • update README with information about Bower requirement
  • refactor handling of trailing margins in asciidoc component
  • drop old themes: iconic, colony, maker, rubygems
  • rework readthedocs theme to match new theme on readthedocs.org
  • add motion theme
  • improve build-stylesheet.sh script
  • declare custom sass functions
  • fix background colors for sidebar & example blocks in asciidoctor theme
  • many minor refinements to styles & scss source

- migrate from Foundation 4 to Foundation 5
- upgrade to Compass 1.0.1
- make stylesheet changes necessary to accomodate upgrades
- add Bower configuration
- update README with information about Bower requirement
- refactor handling of trailing margins in asciidoc component
- drop old themes: iconic, colony, maker, rubygems
- rework readthedocs theme to match new theme on readthedocs.org
- add motion theme
- improve build-stylesheet.sh script
- declare custom sass functions
- fix background colors for sidebar & example blocks in asciidoctor theme
- many minor refinements to styles & scss source
@ggrossetie
Copy link
Member

I will do a code review this week-end 😄

@mojavelinux
Copy link
Member Author

👍

@ggrossetie
Copy link
Member

https://github.com/Huddle/PhantomCSS is looking great, I will try to setup a simple script to automate visual testing.

@SevereOverfl0w
Copy link

I'd be happy to pick up the PhantomCSS testing. Could you give me some pointers on where to put the automated visual testing?

It would be nice to push onto Foundation 6, but there isn't a migration guide that I've seen. I'd be happy to dig in.

I have got asciidoctor to compile with libsass (via node-sass and bower) which means that Ruby can go altogether. This might be something to explore, as libsass is more portable than compass/ruby. I used a compass-mixin library from bower.

@ggrossetie
Copy link
Member

Hello @SevereOverfl0w

I'd be happy to pick up the PhantomCSS testing. Could you give me some pointers on where to put the automated visual testing?

I've done some work with PhantomCSS here: #30
The basic idea was to produce a screenshot of the Asciidoctor User Manual before and after the migration.

During my tests, I've only found one small "regression", so I think it's safe to merge this pull request and work on the next migration (Foundation 5 > Foundation 6). Otherwise we will fall behind quickly 😢

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.

3 participants