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

Nomovok #7

Closed
wants to merge 20 commits into from
Closed

Nomovok #7

wants to merge 20 commits into from

Conversation

timohannukkala
Copy link
Contributor

Few changeses:

  • Qt4 -> Qt5
  • Sorting names can be based on lastname first/firstname first
  • Added addresses for user (work, home, other) - Add - View - Edit
  • Added settings page - that allows to sort contacts -> all users -> that are favorite users -> that have phone number -> that have email address
  • requires chances to libcontacts, qtcontacts-sqlite, nemo-qml-plugin-contacts, qmlcontacts
    (if you want to disable all these changes, except qt4 -> qt5, possible too, no need to pull library changes, just edit little qmlcontacts and fix error messages)

@rburchell
Copy link
Contributor

woah. please read up on git rebase -i (and git commit --amend) to create somewhat of a sane history, please

@@ -0,0 +1,79 @@
#
Copy link
Contributor

Choose a reason for hiding this comment

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

don't fork the packaging for apps - just port it to qt5 (see the other ports for examples)

there's not enough people working on this code to justify having both qt4 and qt5 versions

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok. when I read the history, I think, I did mess up with merging. Sorry, I fix those and try again.


import QtQuick 2.0
import com.nokia.meego 2.0
//import org.nemomobile.qmlcontacts 1.0
Copy link
Contributor

Choose a reason for hiding this comment

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

drop unused imports

@thp
Copy link

thp commented Jan 29, 2015

This has merge conflicts, and has not seen any updates from Robin's feedback. Closing this. Please reopen if you still want to get this in.

@thp thp closed this Jan 29, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants