Closed MarcelRobitaille closed 1 year ago
I like this way better indeed. But something made me not use the file name. I can remember why. I want to test this a bit before merging it.
Could you try to add something for me here in this PR?
When you hover the mouse over the pages on the front end, it would be good to have the file_path
which is the second column of the sch_pages
. Then we are going to have a second piece of information with the tooltip.
But something made me not use the file name. I can remember why.
One consequence of using the file name is that if you use the same hierarchical sheet in multiple places, it will only show up once. This could cause unnecessary diffs too since the reference designators will change between instances.
I only renamed the svgs to match the file name because that's what the frontend is requesting according to the 404s in the network tab of dev tools.
I am not sure how this tooltip information gets to the frontend or how to change it. Could you give me a hint?
I am not sure how this tooltip information gets to the frontend or how to change it. Could you give me a hint?
When using Kicad 5 only, the tooltips are the file path but for Kicad 6/7 they are just the filename.
I have these 2 repos if you want to compare the result.
There is this insert_pages_html
(page 160) function that creates those tooltips.
I thought this would be a related issue, but since it is not something generated here, let's address this in a new PR, later.
I thought this would be a related issue, but since it is not something generated here, let's address this in a new PR, later.
Ok, sounds good
@leoheck if that tooltip is being addressed in a future PR, do you think this can be merged?
I fixed the tooltip, it is already in the master/main (whatever the stupid name is now). I did not merge this because I am not sure it works everywhere and because sometimes it puts messages on my screen. So I am not sure it is working fine yet.
Which kind of messages does it put and can I help to test it more thoroughly?
You have to test on other repositories.
-- Leandro Heck
On Wed, Apr 5, 2023 at 6:13 PM Marcel Robitaille @.***> wrote:
Which kind of messages does it put and can I help to test it more thoroughly?
— Reply to this email directly, view it on GitHub https://github.com/leoheck/kiri/pull/79#issuecomment-1498168975, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJX7YG3QLOLX63VM4YD7OTW7XOBPANCNFSM6AAAAAAWN23ANA . You are receiving this because you were mentioned.Message ID: @.***>
Hey, I am closing this PR. your work was merged externally. Could you check using the main branch if it is still working for you? It looks good to me tho.
Seems to be working for me. Thank you!
Thank you for the donation. Today is my birthday and it was a nice coincidence.
What a nice coincidence. I'm glad I put it off so long then
kicad-cli
exports sheets as<project-name>-<sheet-title>.svg
. The frontend expects<sheet-filename>.svg
.This pull request uses the
sch_pages
file to rename the former into the latter.For example, if I have a project called
Blinky
and a sheet with the filenameled_drivers.kicad_sch
but the nameLED Drivers
,kicad-cli
will save this toBlinky-LED Drivers.svg
instead ofled_drivers.svg
.I think this is what you tried to do with:
but that only works if the title matches the filename.
For me these often do not match. I like to keep my filenames in snake case and my titles with capitalization and spaces.