Review Request: attempt to improve the chat view usability on the plasmoid

[Aleix Pol Gonzalez]

> On April 12, 2012, 9:32 a.m., David Edmundson wrote:
> > plasmoid/org.kde.ktp-chatplasmoid/contents/ui/ChatWidget.qml, line 90
> > <http://git.reviewboard.kde.org/r/104554/diff/1/?file=56370#file56370line90>
> >
> >     Not really a fan of 'magic numbers'.
> >     
> >     Ideally we should take margin from the parent's SVGitem, but I don't think that's doable.
> >     
> >     Is there any other slightly neater way to do it?

This is QML, it's not C++, I think there it makes sense to have such literals, although I don't like them either.

In this case, this margin is to reduce the stress on the UI by adding some space on the sides. I think we want to have it and there's no library for these things like in Harmattan where we had UIConstants.

AFAIU If we don't want these literals in plasma overall, we should create more components/improve the situation overall. In any case that's beyond this patch scope.


- Aleix


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104554/#review12349
-----------------------------------------------------------


On April 12, 2012, 1:37 a.m., Aleix Pol Gonzalez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104554/
> -----------------------------------------------------------
> 
> (Updated April 12, 2012, 1:37 a.m.)
> 
> 
> Review request for Telepathy and David Edmundson.
> 
> 
> Description
> -------
> 
> Tries to address a few problems the chat view has and simplifies the code (a tiny bit).
>  - just reduce opacity when the view is at the end. Having a scroll flickering doesn't feel good and helps reduce the stress the scrollbar adds in such a small UI when it's not needed. I set the opacity to 0.3 but maybe 0 would be better, or not. Don't know.
>  - the chat has two behaviors: a browse mode where the user is looking into the log and doesn't want to be distracted and the user is looking into the newer things on the chat. This patch tries to improve the detection of the current state by making it more relaxed. This way it won't happen that if you write too fast, then everything's messed up. [1]
> 
> Other than that, I tried to port the PlasmaWidgets.LineEdit -> PlasmaComponents.TextField. It's not really possible because at 4.8 we can't know when the user pressed "enter". I guess you want to release 0.4 to work on 4.8 so this is not possible, I think. Maybe it should be backported?
> 
> [1] I'm unsure why we need these both having the screen version, I'll leave it up to the maintainers to decide what to do with this.
> 
> 
> Diffs
> -----
> 
>   plasmoid/org.kde.ktp-chatplasmoid/contents/ui/ChatWidget.qml 740cb34 
> 
> Diff: http://git.reviewboard.kde.org/r/104554/diff/
> 
> 
> Testing
> -------
> 
> talked with myself for a while, once again. a lot of "asd" (and permutations) where written tonight.
> 
> 
> Thanks,
> 
> Aleix Pol Gonzalez
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20120412/07d4d44b/attachment.html>