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

fix url protocol integration under linux #5535

Merged
merged 1 commit into from
Jun 4, 2024

Conversation

buzzhuzz
Copy link
Contributor

@buzzhuzz buzzhuzz commented Jun 1, 2024

Description

Fix URL protocol integration under Linux:

  • fix desktop file creation
  • create separate desktop entry per protocol

Known limitations

Placing OrcaSlicer into directory containing special characters ($, space, /, ^, some others) will break desktop integrations (at least for Ubuntu) due to xdg-open 1.1.3 being not fully compatible with the specifications: it does not comply specifications cannot properly handle quoted ("") paths

Tests

Manual tests for thingverse (cura://) and https://www.printables.com/ (prusaslicer://) under next environments:

  • Ubuntu 24.04 Gnome: Google Chrome 125.0.6422.141, Firefox 126.0.2
  • Ubuntu 24.04 i3wm: Google Chrome 125.0.6422.141, Firefox 126.0.2

I'm using locally built OrcaSlicer, without AppImage so community testing are welcome.

@SoftFever
Copy link
Owner

Awesome!
Thank you

@SoftFever SoftFever merged commit 47ab3a0 into SoftFever:main Jun 4, 2024
12 of 13 checks passed
@buzzhuzz buzzhuzz deleted the dbuzz/linux-url-integration branch June 6, 2024 12:37
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