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 svg instead of png + imagemap #504

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

use svg instead of png + imagemap #504

wants to merge 1 commit into from

Conversation

hboutemy
Copy link
Member

makes updates much easier: hyperlink can be added to source .odf file
then export to svg (select all then limit export to selection) in root directory
then run ./prepare-svg.sh

requires install of svgo to rework the svg exported by LibreOffice: https://svgo.dev/

@slawekjaranowski
Copy link
Member

What with: #482 - with puml looks to easier edit, no additional tools will be required
I simply must to find a time to finish a work ... or someone more can help

@hboutemy
Copy link
Member Author

hboutemy commented Mar 19, 2024

@slawekjaranowski you're right, I forgot about it, sorry

for this simple sources list case, yes, puml can be a good solution even easier to update: I like it

my objective is to rework https://maven.apache.org/ref/4.0.0-alpha-13/ : I don't think I'll be able to do it with puml, unless you prove me wrong
I started here because it is a simpler deployment setup, to test features and check it works as expected

notice: did you get link to pages with your puml test?

@hboutemy
Copy link
Member Author

I started here because it is a simpler deployment setup

Just after writing that sentence, I realize that testing on https://maven.apache.org/plugin-tools-archives/plugin-tools-LATEST/ is even easier to get old and new output at the same time

I'll do the test on plugin-tools, which better represent the type of full need

on this sources image and map, it's an opportunity to try to go ahead in parallel

@@ -8,3 +8,5 @@ test-output
.idea
.DS_Store
.factorypath

/*.svg
Copy link
Contributor

Choose a reason for hiding this comment

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

why are we ignoring svg? Isn't the point of this PR to include SVG files?

@Bukama
Copy link
Contributor

Bukama commented Dec 27, 2024

my objective is to rework https://maven.apache.org/ref/4.0.0-alpha-13/ : I don't think I'll be able to do it with puml, unless you prove me wrong I started here because it is a simpler deployment setup, to test features and check it works as expected

I tried...well...looks..."wild" :(

Current image:
image

PUML try:

image

P.S. relations exactly as on the current one, could be optimized a little bit, but well
P.P.S. No URL added to diagram yet

Copy link
Contributor

@elharo elharo left a comment

Choose a reason for hiding this comment

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

The original is easier to read. Definitely make the font bigger and bolder. I'm trying to make up my mind about straight vs ones with right angles.

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.

4 participants