Re: [PATCH -perfbook] defer: Retouch RCUusecases.svg by Inkscape 1.1.1

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Dec 17, 2021 at 10:17:40AM +0900, Akira Yokosawa wrote:
> On Thu, 16 Dec 2021 07:23:04 -0800, Paul E. McKenney wrote:
> > On Thu, Dec 16, 2021 at 11:06:45PM +0900, Akira Yokosawa wrote:
> >> On Thu, 16 Dec 2021 16:59:13 +0900, Akira Yokosawa wrote:
> >>> Some of texts in the figure are lost when rendered by Inkscape
> >>> 1.1.1 on Fedora 35.
> >>> Those texts are placed far left from the viewBox of the figure
> >>> when opened in Inkscape 1.1.1.
> >>>
> >>> This issue is similar to the one resolved by commit 36b07176210d
> >>> ("defer: Make RCUspacetime.svg compatible with Inkscape 1.0 and
> >>> later").
> >>>
> >>> Ungroup and adjust x-coordinates of those texts by Inkscape 1.1.1.
> >>>
> >>> Resulting SVG is compatible with Inkscape 0.92.x on Ubuntu 18.04
> >>> and Ubuntu 20.04 as well as Inkscape >=1.0.
> >>
> >> Well, Firefox still has trouble with this.... (much better, but
> >> still)
> >>
> >> Paul, please wait for v2.
> > 
> > Idiot here forgot to do the inkscape editing on his Fedora system,
> > apologies!
> > 
> > Let me try that.
> > 
> 
> I'm seeing the refined SVG in Firefox, and it still lacks some of
> texts.
> 
> Now I'm opening the SVG in a text editor...
> 
> Oh, there actually are a dozen of "transform="rotate(90)"
> and "transform="rotate(-90)" directives.
> So you did rotate those texts, didn't you?

When I opened the diagram under Fedora, the text at the tops of many of
the boxes was displaced to the left by a couple of page widths worth.

All I did was go into the top-level group, make like I was going to change
the text of one of the affected text boxes, add a couple of rectangles,
and then delete them.  The idea was to force inkscape to save the diagram.

> For perfbook, web-browser compatibility is not an issue,
> but it might bite you when you reuse this figure in an
> html page.
> 
> And I've now noticed that RCUspacetime.svg is not rendered
> right in Firefox, either.
> 
> Let me see if there is a way to make them browser compatible.

I will pull in your patches and see what it looks like.  I had not
considered it before, but I do agree that having a browser-compatible
.svg is a very good thing, so thank you for checking up on this.

For my part, what I have done is to uninstall inkscape on the problematic
system.  This should help prevent future instances of corruption.  :-/

Again, thank you for checking up on this and debugging it!

> > Ah, and I am expecting to do a release before the end of this year,
> > preferably sooner.  This is not a big deal at all:  Release rather
> > than edition!
> 
> Nice!
> I'll review new contents in rcuusage.tex this weekend.

Looking forward to it!

							Thanx, Paul

>         Thanks, Akira
> 
> 
> > 
> > 							Thanx, Paul
> > 
> >>         Thanks, Akira
> >>
> >>>
> >>> Signed-off-by: Akira Yokosawa <akiyks@xxxxxxxxx>
> >>> ---
> >>> Hi Paul,
> >>>
> >>> So I said you are safe as far as you don't rotate texts.
> >>> Apparently I was wrong!
> >>>
> >>> I'm suspecting the cause of the incompatibility is likely related to
> >>> grouping of texts and doing some operation on the grouped text.
> >>>
> >>> In this retouch, I did ungrouping only.  So you might need to do
> >>> proper grouping if you need further edit.
> >>>
> >>>         Thanks, Akira
> >>> --
> >>>  defer/RCUusecases.svg | 1406 +++++++++++++++++++----------------------
> >>>  1 file changed, 656 insertions(+), 750 deletions(-)
> >>>
> >>> diff --git a/defer/RCUusecases.svg b/defer/RCUusecases.svg
> >> [...]



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux