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

feature for window handling in freamless mode #376

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from
Open

feature for window handling in freamless mode #376

wants to merge 8 commits into from

Conversation

hunt978
Copy link

@hunt978 hunt978 commented Jan 24, 2016

add/update components:

Storage.qml : store global data cross over app sessions
Resizer.qml : right-bottom icon for resizing window size
ActionBar.qml : add window draging(pos) double-click(normal/fullscreen)

the feature is enabled when ApplicationWindows.clientSideDecorations set be true

add/update conponents:

Storage.qml : storage global data cross over all app sessions
Resizer.qml : right-bottom icon for resizing window size
ActionBar.qml : add window draging(pos) double-click(normal/fullscreen)

the feature is enabled when ApplicationWindows.clientSideDecorations set
be true
feature for window handling in freamless mode
@ricardomv
Copy link
Member

@ibeliever isn't this managed in the plugin c++ code?

@iBelieve
Copy link
Member

@ricardomv Not really. Yes, we do use Qt's client-side window decorations. But what this is adding support for is combining the window decorations and the toolbar into one common toolbar.

@ricardomv
Copy link
Member

oh when i saw the resize code i thought it was for the window edge resize.

@@ -6,6 +6,7 @@ ApplicationWindow {
id: demo

title: "Material for QtQuick Demo"
clientSideDecorations : true
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i don't think this should be the default behavior and the indentation is wrong use 4 spaces instead of tabs

@iBelieve iBelieve modified the milestone: Version 0.3 Apr 2, 2016
@hunt978
Copy link
Author

hunt978 commented May 12, 2017

first, sorry for the HUGE delay of the padding works. now i have a problem about what is the best practice to pick up the work. i came up with two solutions :

  1. stay with the old starting point( ver_0.2 ) then fix the errors and/or defects.
  2. merge remove/develop into my local feature/frameless_window_handling and go on.

what should i do?

# Conflicts:
#	src/qmldir
#	src/window/ActionBar.qml
#	src/window/ApplicationWindow.qml

should be always staying with the upstream !
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants