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

OuterWire and Internal Sketches feature #46

Merged
merged 2 commits into from Sep 20, 2018
Merged

OuterWire and Internal Sketches feature #46

merged 2 commits into from Sep 20, 2018

Conversation

easyw
Copy link
Contributor

@easyw easyw commented Sep 18, 2018

  • creation of Outer sketch only when possible (outer wire is closed)
  • option to select color for internal sketch added

- creation of Outer sketch only when possible (outer wire closed)
- option to select color for internal sketch added
@shaise
Copy link
Owner

shaise commented Sep 20, 2018

Hi easyw,

Looks good!
Two things we need to consider, though,

  1. The unfold script is taken from Ulrich. He should be notified with your changes, or next versions will not be compatible.
  2. Why do you leave the "Unfold_sketch"? It is exactly same as "Unfold_sketch_OutWire" (BTW, I think Outline is better then OutWire)

shai

@easyw
Copy link
Contributor Author

easyw commented Sep 20, 2018

  1. The unfold script is taken from Ulrich. He should be notified with your changes, or next versions will not be compatible.

@ulrich1a please have a look at this PR (I also notified him at FC forum)

  1. Why do you leave the "Unfold_sketch"? It is exactly same as "Unfold_sketch_OutWire"

This is not always true... in some case the outline wire is not closed (i.e. when the projection of the unfolded has extra edges on the cut or have internal edges not removed by the unfold script)
outline-sketch
wire-not-closed

(BTW, I think Outline is better then OutWire)

I've updated it 😄

@shaise shaise merged commit 21689cd into shaise:master Sep 20, 2018
shaise added a commit that referenced this pull request Sep 20, 2018
@shaise
Copy link
Owner

shaise commented Sep 20, 2018

Thanks easyw.
I have merged the pull request + did some minor layout changes to the gui.
I will investigate your code some more, perhaps I will use it to replace the current "unfold_sketch"

@easyw
Copy link
Contributor Author

easyw commented Sep 20, 2018

@shaise

  • did some minor layout changes to the gui.

I manually added the Gui code because the ui file is outdated...

@shaise
Copy link
Owner

shaise commented Sep 20, 2018

yes I agree. Perhaps I need to remove it.

@easyw
Copy link
Contributor Author

easyw commented Sep 20, 2018

It would be nice to have it updated... it is easier to contribute

@shaise
Copy link
Owner

shaise commented Sep 20, 2018

The problem is I dont know how to insert a custom widget in qdesigner

@easyw
Copy link
Contributor Author

easyw commented Sep 20, 2018

an option is to create a simple button and then change its style
https://stackoverflow.com/questions/15080404/colouring-a-button-in-qt
Just save the ui and then add this code:

    mycolor = '#ff5733'
    ui.pushButtonColor.setStyleSheet("background: " + mycolor + ";");
    ui.pushButtonColor.setMaximumWidth(32)

UnfoldOptions.zip

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.

None yet

2 participants