-
-
Notifications
You must be signed in to change notification settings - Fork 175
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
Layer system #709
Layer system #709
Conversation
Signed-off-by: Marc Prat Masó <marc.prat.maso@estudiantat.upc.edu>
Here are the build results |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally looks good - definitely needs some doc comments in headers. Also seems like a .png
crept in.
Somewhere we need to document the source of the icons (e.g., README)
Signed-off-by: Marc Prat Masó <marc.prat.maso@estudiantat.upc.edu>
Signed-off-by: Marc Prat Masó <marc.prat.maso@estudiantat.upc.edu>
Here are the build results |
Here are the build results |
avogadro/core/layermanager.h
Outdated
/** save custom data, base save should never be called */ | ||
virtual std::string serialize() | ||
{ | ||
assert(true); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we drop the assert
if it's always true?
The Mac build failure needs |
#include <cassert> | ||
#include <map> | ||
#include <memory> | ||
#include <vector> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#include <string>
Here are the build results |
Signed-off-by: Marc Prat Masó <marc.prat.maso@estudiantat.upc.edu>
Currently translated at 27.0% (323 of 1194 strings) Translation: Avogadro/avogadrolibs Translate-URL: https://hosted.weblate.org/projects/avogadro/avogadrolibs/ru/
Currently translated at 100.0% (1188 of 1188 strings) Translation: Avogadro/avogadrolibs Translate-URL: https://hosted.weblate.org/projects/avogadro/avogadrolibs/eo/
* complex cartoons, and coloring, area select Signed-off-by: Marc Prat Masó <marc.prat.maso@estudiantat.upc.edu>
Signed-off-by: Marc Prat Masó <marc.prat.maso@estudiantat.upc.edu>
…yer_system Signed-off-by: Marc Prat Masó <marc.prat.maso@estudiantat.upc.edu>
Here are the build results |
Signed-off-by: Marc Prat Masó marc.prat.maso@estudiantat.upc.edu
Developer Certificate of Origin
Version 1.1
Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129
Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.
Developer's Certificate of Origin 1.1
By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or
(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or
(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.
(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.
Some examples: