-
Notifications
You must be signed in to change notification settings - Fork 16
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
Tweak UI design, refactor UI control logic, cleanup and reorganize code #261
Conversation
tide/core/resources/SideControl.qml
Outdated
sourceComponent: buttonDelegate | ||
} | ||
// Can't be made a top-level item; this results in unexplained incorrect | ||
// centering of buttons inside the Canevas. |
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.
typo 'Canevas'
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.
done
tide/core/scene/Content.h
Outdated
/** The different types of interaction. */ | ||
enum class Interaction | ||
{ | ||
AUTO, // interaction is enabled automatically (default) |
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.
lower-case values? auto does not work then probably. what does auto interaction mean? depending on content type?
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.
will do, yes that's another thing which could be improved, AUTO is not clear. It serves only for zoomable contents where interaction mode can be changed by tap-and-hold to modify zoom.
tide/core/scene/PixelStreamContent.h
Outdated
@@ -120,7 +124,7 @@ class PixelStreamContent : public MultiChannelContent | |||
const unsigned int version) | |||
{ | |||
serialize_members_xml(ar, version); | |||
_createActions(); // actions are not saved to xml file | |||
_createKeyboard(); // actions are not saved to xml file |
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.
update comment?
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.
in fact this had to be removed, thanks!
@@ -39,6 +39,11 @@ | |||
|
|||
#include "ScreenLock.h" | |||
|
|||
ScreenLockPtr ScreenLock::create() | |||
{ | |||
return ScreenLockPtr(new ScreenLock); |
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.
make_unique/shared?
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.
doesn't work in this case because the constructor is private and make_unique/shared complains about that
tide/wall/WallApplication.cpp
Outdated
_initMPIConnection(); | ||
|
||
_renderController->updateScene(Scene::create(config.surfaces)); | ||
_renderController.reset(new RenderController(*_config, *_provider, |
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.
make_shared/unique
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.
done
No description provided.