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 the vectorised logo #55

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

use the vectorised logo #55

wants to merge 1 commit into from

Conversation

twogee
Copy link
Contributor

@twogee twogee commented Jul 27, 2017

I'd like to remove downloaded.gif, error.gif, evicted.gif and searched.gif as well.
Maybe we could vote on colours for downloaded and evicted 😉 if green is too dark or fuchsia is too garish (I took it because it's one of "safe" colours).

@twogee
Copy link
Contributor Author

twogee commented Aug 1, 2017

@twogee twogee force-pushed the svg-logo branch 2 times, most recently from d5326b5 to 5059785 Compare August 8, 2017 07:10
@twogee
Copy link
Contributor Author

twogee commented Aug 8, 2017

This one seems to break a JIRA record for time of first response: 80000+ hours 😄 Beat that!

@twogee twogee force-pushed the svg-logo branch 8 times, most recently from 1c49df2 to 0b838df Compare August 24, 2017 11:19
@twogee twogee changed the title use vectorised logo use the vectorised logo Sep 3, 2017
@twogee twogee force-pushed the svg-logo branch 2 times, most recently from 865394b to 2e8f52c Compare September 8, 2017 17:44
@twogee twogee force-pushed the svg-logo branch 3 times, most recently from e62e94e to a4a737d Compare September 17, 2017 19:39
@janmaterne
Copy link

ivy-lierre.svg:

  • looks good
  • havent compared, I think asciidoc/images and doc/images are the same?
  • why this name? why not simply logo.svg? (ok, I see: ivy-lierre.png ...) But there is logo.png with same content (only smaller borders)

ivy-report.css

  • seems that here is only different formatting

ivy-report.xsl

  • formatting
  • changed <img ..gif> to
  • translated img/@art->svg//title and img/@title->svg//title
    ! couldnt see the content of the grafik (where is defined that searched/downloaded svg should be used?)

IvyLogo.java

  • havent tested that generated code
  • JavaDoc of the parameters of IvyLogo(int width, int height) are bad ("ditto")
  • I am thinking about the code generation ...
    a) include the flamingo task into our build and remove the generated code (= keep the model (svg) and the generator)
    b) just keep that generated code
    c) reuse the svg (Java reads the svg-resource and interpretes the drawing)
    Because this is one-shot we could go with b) ...

Checked out the code and the report looks fine.
But a simple ivy:report didnt work, I changed build.xml:500 to use explicitly the new xsl
<ivy:report todir="${ivy.report.dir}" xslfile="src/java/org/apache/ivy/plugins/report/ivy-report.xsl"/>
Maybe my local environment ...

@janmaterne
Copy link

Violations of the license header:

  • ivy-lierre.svg (missing, remove the generator comment)
  • IvyLogo (missing)

@twogee
Copy link
Contributor Author

twogee commented Sep 25, 2017

logo.png is for CredentialsUtil; replaced by IvyLogo

CSS contains changes for #logo and adds svg.icon (somehow Firefox cannot read dimensions from embedded SVGs)

XSL generates HTML5 that uses xlink to reuse SVGs

IvyLogo is a one-off conversion of SVG + some manual additions (cleaned casts and added max icon size). Using SVG directly requires Batik, which is an overkill.

Here's the build.xml I used for testing Ivy report

<project name="ivy" default="ivy-report" xmlns:ivy="antlib:org.apache.ivy.ant">
    <target name="init-ivy">
        <taskdef resource="org/apache/ivy/ant/antlib.xml"
                 uri="antlib:org.apache.ivy.ant" classpath="build/artifact/jars/ivy.jar"/>

        <ivy:configure override="true" file="src/example/bintray/ivysettings.xml"/>
    </target>

    <target name="resolve" depends="init-ivy">
        <ivy:resolve organisation="org.hibernate" module="hibernate" revision="3.2.6.ga" transitive="true"
                     inline="true" conf="default" resolveid="hibernate.resolve"/>
    </target>

    <target name="ivy-report" depends="resolve">
        <mkdir dir="build/images"/>
        <ivy:report todir="build/images"
                    conf="default" resolveid="hibernate.resolve"/>
    </target>
</project>

@twogee twogee force-pushed the svg-logo branch 2 times, most recently from cd267a8 to 21d75b3 Compare October 8, 2017 20:34
@twogee twogee force-pushed the svg-logo branch 3 times, most recently from 5a93541 to 771c17e Compare October 23, 2017 04:38
@janmaterne
Copy link

Still open are:

IvyLogo.java

JavaDoc of the parameters of IvyLogo(int width, int height) are bad ("ditto")

test the report
But a simple ivy:report didnt work, I changed build.xml:500 to use explicitly the new xsl
<ivy:report todir="${ivy.report.dir}" xslfile="src/java/org/apache/ivy/plugins/report/ivy-report.xsl"/>
Maybe my local environment ...
--> does the fresh built Ivy use the SVG graphics?

Violations of the license header:

ivy-lierre.svg (header missing, remove the generator comment)
IvyLogo (header missing)

@twogee
Copy link
Contributor Author

twogee commented Jan 9, 2018

Thanks for recapitulating the issues; headers and javadoc corrected.

I tested the xsl using the buildfile provided above (without pointing out the xsl explicitly), so I tend to suspect you're picking up an old version of Ivy.

@twogee twogee force-pushed the svg-logo branch 2 times, most recently from 8d72f1d to e043d2f Compare February 3, 2018 06:55
@twogee twogee mentioned this pull request Feb 9, 2018
@twogee twogee force-pushed the svg-logo branch 2 times, most recently from ad2cd5e to 208fb58 Compare March 3, 2018 11:21
@twogee twogee force-pushed the svg-logo branch 4 times, most recently from c198532 to 165bea3 Compare March 15, 2018 18:43
@twogee twogee force-pushed the svg-logo branch 2 times, most recently from f304bee to ac913f5 Compare March 21, 2018 15:56
@twogee twogee force-pushed the svg-logo branch 2 times, most recently from 646d322 to 70eca12 Compare August 29, 2018 06:52
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