[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-distro-pkg-dev
Subject:    [rfc][icedtea-web] fixinf of desktop icon behaviour
From:       omajid () redhat ! com (Omair Majid)
Date:       2012-12-31 14:17:00
Message-ID: 50E19E5C.1010103 () redhat ! com
[Download RAW message or body]

On 12/29/2012 01:32 PM, Jiri Vanek wrote:
> On 12/27/2012 07:04 PM, Omair Majid wrote:
>> On 12/21/2012 11:38 AM, Adam Domurad wrote:
>>> On 12/21/2012 11:05 AM, Jiri Vanek wrote:
>>>> Hi! This is fix for
>>>> http://icedtea.classpath.org/bugzilla/show_bug.cgi?id=725 - JNLP
>>>> applications will prompt for creating desktop shortcuts every time
>>>> they are run
>>>>
>>>> When I have seen Omairs original patch it was too much complicated, So
>>>> I want at least to try this one.
>>>
>>> Do you know why Omair used explicit locking? I couldn't quite figure
>>> it out
>>>
>>
>> My first patch kept a record of all desktop entries that icedtea-web
>> created in a file. This meant that if a user deleted the desktop icon,
>> icedtea-web wouldn't prompt for the creation of that icon again.
>>
> 
> I was thinking about it in vice-versa. If he had  already created icon,
> and then he delted it. And some time later he launch application again,
> I would expect he would like to be asked again.

I don't have strong feelings about this either way. Some user testing
might prove insightful.

> However, when he will refuse to create an desktop icon then he should
> not be asked again.. that is *probably* true, but I'm not so sure... And
> to achieve this something like your previous approach is needed.

This sounds sensible to me too.

> So what do you think about combination of my and yours approach?

Sounds fine to me.

I think it would be best to get the behaviour that we are sure is needed
(what your initial patch does) into icedtea-web, and then we can try and
figure out what to do with the corner cases where the right thing to do
is not as obvious.

Cheers,
Omair


-- 
PGP Key: 66484681 (http://pgp.mit.edu/)
Fingerprint = F072 555B 0A17 3957 4E95  0056 F286 F14F 6648 4681

[prev in list] [next in list] [prev in thread] [next in thread]