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

rearrange library symbols (WIP) #592

Closed
wants to merge 22 commits into from

Conversation

felix-salfelder
Copy link
Member

followup to #556 (hotfix merged). trying to rearrange library symbols.

this definitely needs work. there's some overlap with other PRs requiring symbol.h

andresmmera and others added 22 commits November 6, 2016 01:14
This PR fixes both issues Qucs#550 and Qucs#553. It was found that QucsLib tool
didn't update neither the default symbol nor the name/library of the
selected component
LoadDefaultSymbol() was removed and its code was placed where the
function was called.
The code for reloading the default symbol was moved to SymbolWidget
class.
(did not compile, some redefinition?!)
don't know what DragNDropWidth is..
Before this commit, an error related to QucsSettings object remained
there...

It seems that, apart from the QWidget issue, the rest of the code do not
cause errors.
now works with
$ make
$ QUCS_LIBRARY=../qucs-lib/library/ qucs-lib/qucslib
and remove some non-symbol text from symbols
some symbols (diode, JFET) dont paint (why?)
@guitorri guitorri added this to the 0.0.20 milestone Nov 23, 2016
@guitorri guitorri changed the base branch from release-0.0.19 to develop January 22, 2017 13:22
@guitorri
Copy link
Member

Moved base to develop.

@guitorri
Copy link
Member

guitorri commented Oct 6, 2017

It seems some work is still needed. Moving it forward.

@guitorri guitorri modified the milestones: 0.0.20, 0.0.21 Oct 6, 2017
@felix-salfelder felix-salfelder mentioned this pull request Jul 22, 2020
38 tasks
@felix-salfelder
Copy link
Member Author

obsolete with #898

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants