Re: Proposal: Drop support for Internet Explorer

From: Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: Neel Patel <neel(dot)patel(at)enterprisedb(dot)com>, Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com>, Darren Duncan <darren(at)darrenduncan(dot)net>, pgAdmin Support <pgadmin-support(at)postgresql(dot)org>, pgadmin-hackers <pgadmin-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Proposal: Drop support for Internet Explorer
Date: 2020-04-14 15:47:55
Message-ID: CANxoLDev2jgC2dnSj5ALSfDoLPg6AHmq359bAcwYsLqJ4Ae9qw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers pgadmin-support

Thanks, patch applied.

On Tue, Apr 14, 2020 at 8:17 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:

> Ooops. Thanks for catching that. Here it is.
>
> On Tue, Apr 14, 2020 at 3:45 PM Neel Patel <neel(dot)patel(at)enterprisedb(dot)com>
> wrote:
>
>> Hi Dave,
>>
>> Looks like patch is missing in attachment.
>>
>> Thanks,
>> Neel Patel
>>
>>
>> On Tue 14 Apr, 2020, 6:53 PM Dave Page, <dpage(at)pgadmin(dot)org> wrote:
>>
>>> Here's an updated patch that gives a slightly different message if the
>>> browser is unknown vs. unsupported/deprecated. As with the previous patch,
>>> the check can be disabled in the config.
>>>
>>> On Fri, Apr 10, 2020 at 5:07 AM Khushboo Vashi <
>>> khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
>>>
>>>>
>>>>
>>>> On Thu, Apr 9, 2020 at 11:57 PM Darren Duncan <darren(at)darrenduncan(dot)net>
>>>> wrote:
>>>>
>>>>> The patch looks good as much as I understand it, but this raises an
>>>>> important
>>>>> question:
>>>>>
>>>>> How should one best handle minority browsers that may be completely
>>>>> modern but
>>>>> you may not specifically know about them? Such as the newer crop of
>>>>> browsers
>>>>> that emphasize stronger privacy or may have fewer identifiers?
>>>>>
>>>>> While going on a whitelist as the patch essentially does for known
>>>>> good browsers
>>>>> is conservative, I feel that an alteration would be good.
>>>>>
>>>>> I propose dividing the browsers/environments into 3 categories, which
>>>>> are
>>>>> recognized-supported, recognized-unsupported, and unrecognized.
>>>>>
>>>>
>>>>> So the unsupported older versions of supported browsers get a stronger
>>>>> message
>>>>> encouraging a browser switch as they are recognized as unsupported,
>>>>> while
>>>>> unrecognized browsers get a different weaker message saying they
>>>>> weren't
>>>>> recognized so we can't determine if they'd work; both can point to the
>>>>> list of
>>>>> known supported browsers.
>>>>>
>>>>> I do agree with this suggestion.
>>>>
>>>>
>>>>> Related to this, there could be an application toggle that affects the
>>>>> unrecognized category where users can basically say, yes I understand
>>>>> you don't
>>>>> recognize this browser, please hide the warning, or something like
>>>>> that.
>>>>>
>>>>> Also, it probably goes without saying, but the code/templates will
>>>>> need to be
>>>>> structured in such a way that the warning message uses about plain as
>>>>> possible
>>>>> HTML so that if the browser doesn't support displaying the UI in
>>>>> general it can
>>>>> at least display the message.
>>>>>
>>>>> -- Darren Duncan
>>>>>
>>>>> On 2020-04-09 4:36 a.m., Dave Page wrote:
>>>>> > Hi
>>>>> >
>>>>> > On Thu, Apr 9, 2020 at 12:26 AM Darren Duncan wrote:
>>>>> >
>>>>> > If its hard to know how many people are actually using Internet
>>>>> Explorer:
>>>>> >
>>>>> > You could make the next release of pgAdmin display a message
>>>>> occasionally to
>>>>> > users of Internet Explorer saying that Internet Explorer will no
>>>>> longer be
>>>>> > officially supported in a future version, and when that version
>>>>> comes the
>>>>> > message says now no longer supported.
>>>>> >
>>>>> > You can then see how many people contact you about this to
>>>>> express concern.
>>>>> >
>>>>> >
>>>>> > Good idea. I've hacked up a patch to warn users if they're using a
>>>>> deprecated or
>>>>> > unsupported browser.
>>>>> >
>>>>> > CCing Akshay for a review :-)
>>>>> >
>>>>> > --
>>>>> > Dave Page
>>>>> > Blog: http://pgsnake.blogspot.com
>>>>> > Twitter: @pgsnake
>>>>> >
>>>>> > EnterpriseDB UK: http://www.enterprisedb.com
>>>>> > The Enterprise PostgreSQL Company
>>>>>
>>>>>
>>>>>
>>>>>
>>>
>>> --
>>> Dave Page
>>> Blog: http://pgsnake.blogspot.com
>>> Twitter: @pgsnake
>>>
>>> EnterpriseDB UK: http://www.enterprisedb.com
>>> The Enterprise PostgreSQL Company
>>>
>>
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>

--
*Thanks & Regards*
*Akshay Joshi*

*Sr. Software Architect*
*EnterpriseDB Software India Private Limited*
*Mobile: +91 976-788-8246*

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Akshay Joshi 2020-04-14 15:48:11 Re: RM4292 - Dark mode support for Windows/macOS
Previous Message Akshay Joshi 2020-04-14 15:46:31 pgAdmin 4 commit: Warn the user if an unsupported, deprecated or unknow

Browse pgadmin-support by date

  From Date Subject
Next Message Michael Benjamin 2020-04-14 17:09:32 Re: Export Error
Previous Message Dave Page 2020-04-14 14:46:56 Re: Proposal: Drop support for Internet Explorer