Reply
Thread Tools
Posts: 22 | Thanked: 51 times | Joined on Dec 2014
#591
Events view controls should be included in official releases IMO! Great work.
Just one thing I'd change is that the controls would be in the center and notifications on top. I looked at the patch and tried something but with my minimal experience I only managed to get everything overlapping on top of each other on the top

Any help?
 
coderus's Avatar
Posts: 6,436 | Thanked: 12,701 times | Joined on Nov 2011 @ Ängelholm, Sweden
#592
@synt events is list. we can't place it "to center", because list is scrolling, and there are no center at all
by same reason we can't put it to bottom, when list is empty bottom at top, when list is filled bottom is not visible before you scrolling it
__________________
Telegram | Openrepos | GitHub | Revolut donations
 
Posts: 752 | Thanked: 2,808 times | Joined on Jan 2011 @ Czech Republic
#593
Originally Posted by coderus View Post
@synt events is list. we can't place it "to center", because list is scrolling, and there are no center at all
by same reason we can't put it to bottom, when list is empty bottom at top, when list is filled bottom is not visible before you scrolling it
What about using bottom DockedPanel, then? It works quite well for controls in the Media Player.

By the way, have you thought about making it landscape-friendly (letting the sliders stretch and adjusting the number of items in the grid row when in landscape)?

Just a suggestion
 

The Following 2 Users Say Thank You to nodevel For This Useful Post:
coderus's Avatar
Posts: 6,436 | Thanked: 12,701 times | Joined on Nov 2011 @ Ängelholm, Sweden
#594
@nodevel of course we can remake whole eventsview page, but no, not this time
__________________
Telegram | Openrepos | GitHub | Revolut donations
 
Posts: 752 | Thanked: 2,808 times | Joined on Jan 2011 @ Czech Republic
#595
Originally Posted by coderus View Post
@nodevel of course we can remake whole eventsview page, but no, not this time
Can I fork it then?

My first suggestion was just an idea to solve the 'move it to bottom' problem, not something I would need.
IMHO the second one is just a matter of ~2 lines of code, nothing to re-make.

With Math.floor(number of columns times the landscape/portrait ratio), one wouldn't even have to make any additional setting.

PS: Not sure how you feel about pull requests.
 
coderus's Avatar
Posts: 6,436 | Thanked: 12,701 times | Joined on Nov 2011 @ Ängelholm, Sweden
#596
@nodevel you can create patch with different name based on my patch.
Pull request to extend current patch is ok, not for a different patch changing everything
and in landscape enevtsview working crazy, i didn't investigated it, just used Screen.width instead of parent width as workaround
__________________
Telegram | Openrepos | GitHub | Revolut donations
 

The Following User Says Thank You to coderus For This Useful Post:
Posts: 46 | Thanked: 11 times | Joined on Jun 2013
#597
Can someone tell me where to find some documentation to learn how to develop patches?
 
coderus's Avatar
Posts: 6,436 | Thanked: 12,701 times | Joined on Nov 2011 @ Ängelholm, Sweden
#598
i'm generating patches from git commits
__________________
Telegram | Openrepos | GitHub | Revolut donations
 
Posts: 7 | Thanked: 7 times | Joined on Dec 2014
#599
Mmmh.. Some problems on my side.
Every installation passes, but it seems impossible to activate it in PM (setting or ui)
Is there any possibility to have a detailled output of the problem by patchmanager ? ( some journal or else.)
 
Posts: 55 | Thanked: 34 times | Joined on Jun 2014
#600
Originally Posted by asa View Post
Anyone else getting this warning?
https://public.bn1304.livefilestore....1223233930.jpg

I have the lastest version.
Am I the only one having this?



@Baspar
Have you tried this
http://talk.maemo.org/showpost.php?p...&postcount=559
 
Reply

Tags
patchmanager, sailfish os

Thread Tools

 
Forum Jump


All times are GMT. The time now is 07:47.