<div dir="ltr">Hi James,<br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 2, 2015 at 9:58 AM, James Cameron <span dir="ltr"><<a href="mailto:quozl@laptop.org" target="_blank">quozl@laptop.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Summary: counter proposal, use modal alerts.  +Subject-tag [DESIGN].<br>
<br>
On Sun, Nov 01, 2015 at 02:59:46PM +1100, Sam P. wrote:<br>
> In sugar, some subsystems need to show an alert to the user<br>
> wherevever they are.  Right now, we have 2 systems; the max<br>
> activities limit and Quozl's force power off alerts.<br>
<br>
For readers who haven't seen this feature yet, think of it as a logout<br>
failure alert, caused by an activity not responding to session<br>
manager.<br>
<br>
It is rare enough that I'm fine with it disappearing if the user<br>
changes view.  They can change back to see it again.<br>
<br>
The maximum activities limit isn't generally used, so I'm not worried<br>
about it either.<br>
<br>
> Currently these are alerts that show in some places but not others. <br>
> They are not system wide alerts, so the user may not always see<br>
> them.   Plus there is too much code to make sure they are visible in<br>
> many places.  We also have a completely different implantation for<br>
> the full journal alert, which takes up the whole screen.<br>
<br>
Yes, this ModalAlert is different because the journal activity starts<br>
without explicit user action, and by the time it does this the focus<br>
may have moved away from the home view.<br>
<br>
It is the only alert that isn't triggered by the learner, and the<br>
problem of full journal really does stop the learner, so I'm fine<br>
with it being entirely different.<br>
<br>
> It is a mess IMO.<br>
<br>
Yes, but not a DESIGN issue, it's a code issue.  See below.<br>
<br>
> I think that if we come up with a ui for this, then we can hopefully<br>
> get rid of duplicated code!<br>
<br>
It is both the duplicated code and the inconsistent methods by which<br>
alerts are displayed.  Some view classes have inconsistent alert<br>
handling.<br>
<br>
But that isn't actually a learner problem.  It's a messy code problem.<br>
<br>
There are also alerts that do not need to be global;<br></blockquote><div><br></div><div>Yeah I agree.  I don't think that we should change those alerts :)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
- journal, to confirm entry erase,<br>
<br>
- journal, to confirm batch operations, and report progress,<br>
<br>
- home view list, to confirm activity erase,<br>
<br>
- control panel accept, to restart,<br>
<br>
- school server registration,<br>
<br>
- journal, volume errors, caused by copying errors,<br>
<br>
- downgrading an activity,<br>
<br>
- view source, to confirm duplicating, and report progress,<br>
<br>
- view source, duplicating already duplicated activity,<br>
<br>
- activities, such as Chat join, and Browse downloads.<br>
<br>
> To get the conversation started, what about a simple idea.  Maybe we<br>
> could just have the alert (as in the long ones with buttons on the<br>
> end) pop up along the top of the screen and disable the frame.  We<br>
> could give it a red border of a massive shadow (like metacity loves)<br>
> or something.  This would block the user from accessing their<br>
> activity toolbar, which would probably make them look at it :)  See<br>
> [1]<br>
<br>
Thanks, but no, I dislike the idea.<br>
<br>
Here's what I think should be done;<br>
<br>
- generalise the ModalAlert; at the moment it is journal specific,<br>
<br>
- use the ModalAlert for the maximum activities limit; this will<br>
  fix your concern in opening paragraph,<br>
<br>
- extend the ModalAlert to include a timeout, as a TimeoutModalAlert,<br>
  in a similar fashion to how TimeoutAlert is implemented,<br>
<br>
- use the TimeoutModalALert for the logout failure message; this will<br>
  fix your concern in opening paragraph,<br>
<br>
- extend the maximum activities limit to show a list of activities and<br>
  let the learner close one, </blockquote><div><br></div><div>Yes, that sounds like a great idea.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
- simplify the non-modal alert calls and UI object marshalling.<br></blockquote><div><br></div><div>That seems kind of separate to me.  What do you mean?</div><div><br></div><div>Thanks,</div><div>Sam</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
What do you think?<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
James Cameron<br>
<a href="http://quozl.linux.org.au/" rel="noreferrer" target="_blank">http://quozl.linux.org.au/</a><br>
</font></span></blockquote></div><br></div></div>