View Single Post
Posts: 239 | Thanked: 194 times | Joined on Jul 2010 @ Amsterdam
#18
Originally Posted by kanishou View Post
- The splashscreen you added has some odd pattern in the background, possibly related to dithering but I'm not sure.
The odd pattern is shown on all of my screens which don't have a color defined (transparent). I'll address this later by having every screen start with a root Rectangle which by default has a white background color as opposed to transparent which it is now. The transparent screen is the cause of these patterns. On my N950 it's on every screen though. I guess your N9 does not have this on the other screens?

Originally Posted by kanishou View Post
- Now you removed the menu from the title, I still think it would be nice though if the title would show the name of the active list and be a menu that opens the list editor. You could have a default list called "EasyList" so it would show the application name by default. Also the toolbar button could be removed then.
I still like to see the app name EasyList so I've added the list name behind it.

Originally Posted by kanishou View Post
- There is no pressed feedback on the list items, which would be useful to see which item you are long-tapping on.
I've added a hover color now.

Originally Posted by kanishou View Post
- I am now missing a "rename" function for the list editor.
I've added the rename function.

Originally Posted by kanishou View Post
- Now that it is not really necessary to select a list in the list editor for any functionality, I think it would be nicer if selecting a list would immediately switch back to the list view. The back button could then also be removed from this toolbar. The delete button IMO should lead to a view which allows multiple selection of items to delete.
Selecting a list will now also go back to the List items page.

Originally Posted by kanishou View Post
- In the "name the new list" sheet, the text entry should have full width, with the label above it. You can see an example of it here:

http://www.developer.nokia.com/swipe...xt_fields.html ("Text field title #1").

Also the "Name your new list" text is not necessary or should be the text field title instead.
I've addressed this now.

Originally Posted by kanishou View Post
- In the settings dialog, instead of having "lock portrait" and "lock landscape" options for orientation, I suggest to have a splitbutton with three options: "Automatic", "Portrait", and "Landscape". Then you can also remove the help button. You can see an example of such a split button here:

http://www.developer.nokia.com/swipe...s/Buttons.html (the one in the lower right).
Good suggestion. I'll look into this soon.

Originally Posted by kanishou View Post
- The styling of the switch buttons is a bit wrong (wrong handle graphic), but I guess that this is a qt-components bug if you are not doing something special.
It's a normal Switch button. I've done nothing special with it...

Originally Posted by kanishou View Post
- The styling of the headers in the settings page is a bit wrong, not a big deal but I just happened to implement the correct styling a while ago, so I can just as well mention it. I think this should be a standard component, but I don't know if it will ever be. So you can use this as the section.delegate in your listview (or standalone item whatever you prefer):

Code:
Item {
                width: parent.width
                height: 40
                Text {
                    id: headerLabel
                    anchors.right: parent.right
                    anchors.bottom: parent.bottom
                    anchors.rightMargin: 8
                    anchors.bottomMargin: 2
                    text: section
                    font.bold: true
                    font.pointSize: 18
                    color: theme.inverted ? "#4D4D4D" : "#3C3C3C";
                }
                Image {
                    anchors.right: headerLabel.left
                    anchors.left: parent.left
                    anchors.verticalCenter: headerLabel.verticalCenter
                    anchors.rightMargin: 24
                    source: "image://theme/meegotouch-groupheader" + (theme.inverted ? "-inverted" : "") + "-background"
                }
(please excuse the messy indentation)
Many thanks for this code. It will surely come in handy. However I have different plans for my Settings page. It's not a list because there will be different options in the future which can't be drawn by a single delegate.

Originally Posted by kanishou View Post
- There should be no "Back" item in the application menu of the about view. Actually there probably should be no application menu in that view in the first place, it's a bit unnecessary.
I've addressed this issue.

Again. Many thanks for your comments. I've found a bug in 0.0.10 which is quite critical. So I'm going to release 0.0.11 very soon with the fix.
__________________
Please give or donate your 2 cents to help me keep on going.

Last edited by Willem Liu; 2011-08-27 at 19:46.