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 Unicode glyphs or SVG data URLs instead of bitmaps #60

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

twogee
Copy link
Contributor

@twogee twogee commented Aug 30, 2017

I put this up for review in case I missed some page that used the removed icons.

@@ -179,11 +180,11 @@
</target>

<target name="init-asciidoctor" depends="init-asciidoctor-extensions" unless="skip.doc">
<ivy:cachepath pathid="asciidoctor.path" organisation="org.asciidoctor" module="asciidoctor-ant" revision="1.5.4" inline="true"/>
<ivy:cachepath pathid="asciidoctor.path" organisation="org.asciidoctor" module="asciidoctor-ant" revision="1.5.3" conf="default" inline="true"/>
<taskdef uri="antlib:org.asciidoctor.ant" resource="org/asciidoctor/ant/antlib.xml" classpathref="asciidoctor.path">
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of downgrading the version, I think we should try and figure out whatever the issue is with 1.5.4

Copy link
Contributor Author

@twogee twogee Aug 31, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very well, here's the output from 1.5.4:

.../ant-ivy/build-release.xml:202: org.jruby.exceptions.RaiseException: (LoadError) no such file to load -- asciidoctor
	at org.jruby.RubyKernel.require(org/jruby/RubyKernel.java:1040)
	at RUBY.require(.../.ivy2/cache/org.asciidoctor/asciidoctor-ant/jars/asciidoctor-ant-1.5.4.jar!/META-INF/jruby.home/lib/ruby/shared/rubygems/core_ext/kernel_require.rb:54)
	at RUBY.(root)(<script>:9)

@twogee
Copy link
Contributor Author

twogee commented Sep 3, 2017

I changed back the version of asciidoctor-ant to 1.5.4 and added a stab at IVY-450. The current description does not indicate what should happen when the parent is collapsed, so I assume that one may as well use breadcrumbs rather than visual cues from the menu.

@twogee
Copy link
Contributor Author

twogee commented Sep 3, 2017

Looks like I've beaten time to first response record again: 90000+ hours 😃

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.

2 participants