b4j4 / indicator-kdeconnect

AppIndicator for KDE Connect
GNU Lesser General Public License v2.1
372 stars 38 forks source link

- Disable send button if there is no phone number and message text #55

Closed OzymandiasTheGreat closed 7 years ago

OzymandiasTheGreat commented 7 years ago

I'm thinking I should also change data location on disk to something like ~/.local/share/indicator-kdeconnect/sms and the name of the app on the authorization page to KDE Connect Indicator to avoid confusion. If you agree, don't merge yet. I'll make the changes asap.

b4j4 commented 7 years ago

I'm sorry, i accept the commit via smartphone and i don't read your comment. You should made the change to the referred data location and the name of the page, since this passes to be make part of indicator-kdeconnect not make sense have other name.

One more thing, lis really nice to have a sms and character count like the old SMS sender.

captura de ecra de 2017-04-19 20-53-35

Maybe the algorithm that i use on the last one help you.

captura de ecra de 2017-04-19 20-58-23