Pattismith - Githubissues

veskuh / Tweetian

A feature-rich Twitter app for smartphones developed using Qt/QML

GNU General Public License v3.0

50 stars 26 forks source link

Pattismith #134

Closed ghost closed 10 years ago

ghost commented 10 years ago

I want a simpler way to "scroll to top". Moving hand to icon at bottom fells awkward.

So I tried to implement similar concet as tidings app but a bigger icons since I think original is to small.

Problem is I am not sure this solution still not the best... We need testers...

First FIXME; The icon I made is just rescaled from original jolla arrow and could be done better
Second FIXME: for some uknown reason it does not fully scroll to top using flickered.scrollToTop() seems like a silica bug? Raise condtion since onClicked triggered and still scrolling?

However with some more work I think this could be merged. Maybe some icon guru can make a better icon :)

Morpog commented 10 years ago

There you go: http://abload.de/img/arrow-up8sd1j.png

ghost commented 10 years ago

thanks for icon :)

ghost commented 10 years ago

Btw after some thinking maybe we should have another icon since we scroll to top it should be double sign << or: |< (rotated ofcourse:)

Morpog commented 10 years ago

http://abload.de/img/arrow-up2lvk7k.png or http://abload.de/img/arrow-up3dtkvp.png

ghost commented 10 years ago

added the one with >>

veskuh commented 10 years ago

Please, use descriptive title in PR. I prefer to collect change log from closed PR titles and thus want them to be descriptive of the change.

veskuh commented 10 years ago

I'd prefer having button for going up and for going down. The visibility should be so that when you start scrolling up the button to go up would become visible and when you stop after timeout the button disappears, and same for going down.

ghost commented 10 years ago

pushed a reworked one as a new mergerequest named it gototop. This merge request is deprecAtad