Custom cursors and high dpi (Windows and MacOS testers needed)
Forum rules
Be nice to others! Respect the FreeCAD code of conduct!
Be nice to others! Respect the FreeCAD code of conduct!
-
- Posts: 100
- Joined: Sat Jun 27, 2020 9:06 am
- Location: Dorset, England
Re: Custom cursors and high dpi (Windows and MacOS testers needed)
.. and if anyone's interested I've built my own app (currently windows only but should able to compile for Ubuntu) to automate most of the cleanup process:
Re: Custom cursors and high dpi (Windows and MacOS testers needed)
Wait, what?mikeprice99 wrote: ↑Thu Jul 30, 2020 11:27 am .. and if anyone's interested I've built my own app (currently windows only but should able to compile for Ubuntu) to automate most of the cleanup process:
Since you know about SVG, maybe you can take a look at the open pull request #3749. I basically move the functions outside the main getSVG function, so each of them can be tested individually. I did not change the logic of the code significantly, I mostly cleaned up a few things.
This code is responsible for generating the SVG for export from the Draft Workbench, through the importSVG module. The getSVG code is used also indirectly by TechDraw in order to project specific objects from Draft/Arch (TechDraw_ArchView).
Always add the important information to your posts if you need help. Also see Tutorials and Video tutorials.
To support the documentation effort, and code development, your donation is appreciated: liberapay.com/FreeCAD.
To support the documentation effort, and code development, your donation is appreciated: liberapay.com/FreeCAD.
Re: Custom cursors and high dpi (Windows and MacOS testers needed)
I compiled the new branch and saw the new cursors. They look good.mikeprice99 wrote: ↑Sat Jul 25, 2020 10:51 am If you do want the cursors recreated in svg, it's now taking me less than 5 minutes per cursor.
...
The Sketcher_Trimming (Sketcher_Pointer_Trimming.svg) and Sketcher_Extend (Sketcher_Pointer_Extension.svg) tools, however, look a bit weak because they use a dashed line. Maybe a thicker line, or less segments, could be used.
The Sketcher CreateSlot (Sketcher_Pointer_Slot.svg) looks like an ellipse and not a slot, so this one should be modified. See my mockup.
Always add the important information to your posts if you need help. Also see Tutorials and Video tutorials.
To support the documentation effort, and code development, your donation is appreciated: liberapay.com/FreeCAD.
To support the documentation effort, and code development, your donation is appreciated: liberapay.com/FreeCAD.
-
- Posts: 100
- Joined: Sat Jun 27, 2020 9:06 am
- Location: Dorset, England
Re: Custom cursors and high dpi (Windows and MacOS testers needed)
@vocx I agree!. This should fix it. (Also changed crosshairs to white as requested.)
With regard to the PR you mentioned, are there any specific problems? If so, I'd recommend a separate topic in the forum.
With regard to the PR you mentioned, are there any specific problems? If so, I'd recommend a separate topic in the forum.
- Attachments
-
- cursor_slot_new.svg
- (464 Bytes) Downloaded 53 times
Re: Custom cursors and high dpi (Windows and MacOS testers needed)
There are no specific problems, I was just hoping you would check the SVG generation is correct. That code was not written by me (I haven't read the SVG specifications), but by other authors over the years, so I was just hoping that if you see a problem that can be fixed easily then you let me know. But if you want to make more improvements then go ahead (but maybe you would also need to become more familiar with the Draft Workbench code).mikeprice99 wrote: ↑Thu Jul 30, 2020 4:54 pm ...
With regard to the PR you mentioned, are there any specific problems? If so, I'd recommend a separate topic in the forum.
Always add the important information to your posts if you need help. Also see Tutorials and Video tutorials.
To support the documentation effort, and code development, your donation is appreciated: liberapay.com/FreeCAD.
To support the documentation effort, and code development, your donation is appreciated: liberapay.com/FreeCAD.
-
- Posts: 100
- Joined: Sat Jun 27, 2020 9:06 am
- Location: Dorset, England
Re: Custom cursors and high dpi (Windows and MacOS testers needed)
I wouldn't want to touch it before becoming very familiar with draft workbench.
However, skimming through the original getSVG.py file I did note that in just 4 places stroke-width has been specified in pixels. No other values are specified in pixels. This could cause unexpected line widths.
Re: Custom cursors and high dpi (Windows and MacOS testers needed)
Probably opening a ticket to track this would be important.mikeprice99 wrote: ↑Thu Jul 30, 2020 5:23 pm However, skimming through the original getSVG.py file I did note that in just 4 places stroke-width has been specified in pixels. No other values are specified in pixels. This could cause unexpected line widths.
Alone you go faster. Together we go farther
Please mark thread [Solved]
Want to contribute back to FC? Checkout:
'good first issues' | Open TODOs and FIXMEs | How to Help FreeCAD | How to report Bugs
Please mark thread [Solved]
Want to contribute back to FC? Checkout:
'good first issues' | Open TODOs and FIXMEs | How to Help FreeCAD | How to report Bugs
Re: Custom cursors and high dpi (Windows and MacOS testers needed)
Let us keep discussions about Draft SVG here, Improvements in SVGmikeprice99 wrote: ↑Thu Jul 30, 2020 4:54 pm ...
With regard to the PR you mentioned, are there any specific problems? If so, I'd recommend a separate topic in the forum.
Always add the important information to your posts if you need help. Also see Tutorials and Video tutorials.
To support the documentation effort, and code development, your donation is appreciated: liberapay.com/FreeCAD.
To support the documentation effort, and code development, your donation is appreciated: liberapay.com/FreeCAD.
Re: Custom cursors and high dpi (Windows and MacOS testers needed)
New sketcher slot icon incorporated. This PR is getting larger and larger. I'm afraid it's getting harder to review. Would you suggest I create a separate PR or continue improvements in this one?
Further changes are going to be more intrusive. I don't want them to block the work we've already done. Is there a list of reviewers/contributors with push access I can ping? Is there a more interactive channel of communication? Say, a quick video chat so I can walk you through the changes and demonstrate how it works? Anything I can do to speed up the review...
Further changes are going to be more intrusive. I don't want them to block the work we've already done. Is there a list of reviewers/contributors with push access I can ping? Is there a more interactive channel of communication? Say, a quick video chat so I can walk you through the changes and demonstrate how it works? Anything I can do to speed up the review...
Re: Custom cursors and high dpi (Windows and MacOS testers needed)
This PR should still be small compared to some other ones. It should be a problem if your commits are ordered/squashed correctly. I didn't check but typically this one should have go as a 'fixup' of the one where you introduced the previous slot icon, so the double change is erased.
@vocx already explained PR can take weeks/months to be merged. It depends on reviewers/maintainers spare time and willing. Moreover IMO there are very more important PRs in the queue, especially ones that shall be merged before 0.19 is released.
Further changes are going to be more intrusive. I don't want them to block the work we've already done. Is there a list of reviewers/contributors with push access I can ping? Is there a more interactive channel of communication? Say, a quick video chat so I can walk you through the changes and demonstrate how it works? Anything I can do to speed up the review...
If you still have time and energy, you can go through the bug tracker and help fixing, or start to improve in other fields.