public inbox for [email protected]  

help / color / mirror / Atom feed

From: Neel Patel <[email protected]>
To: Dave Page <[email protected]>
Cc: Khushboo Vashi <[email protected]>
Cc: Darren Duncan <[email protected]>
Cc: pgAdmin Support <[email protected]>
Cc: pgadmin-hackers <[email protected]>
Cc: Akshay Joshi <[email protected]>
Subject: Re: Proposal: Drop support for Internet Explorer
Date: Tue, 14 Apr 2020 20:15:12 +0530
Message-ID: <CACCA4P3DYZJjx+_zNxhHfjzftN7no9QAvwWfyzCsBa+HjoMspg@mail.gmail.com> (raw)
In-Reply-To: <CA+OCxozTYRaa12yA95P58FeYDZ1MPLhJ=KHLqR_DuPBPWu_44w@mail.gmail.com>
References: <CA+OCxowooZqhT3BBLo5KshKjWJZNN7ERQydBmUrRLz=n+1LQKA@mail.gmail.com>
	<[email protected]>
	<CA+OCxowbi+rOK6gxfCd7d9+Dr3Wka27aRmAh7g7=sC1QdQhGqg@mail.gmail.com>
	<[email protected]>
	<CAFOhELcYLbVTW-FM+VHTzPTW86qP-zj+Cc9-Yh+JsPU3w7=3Pg@mail.gmail.com>
	<CA+OCxozTYRaa12yA95P58FeYDZ1MPLhJ=KHLqR_DuPBPWu_44w@mail.gmail.com>

Hi Dave,

Looks like patch is missing in attachment.

Thanks,
Neel Patel


On Tue 14 Apr, 2020, 6:53 PM Dave Page, <[email protected]> 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 <
> [email protected]> wrote:
>
>>
>>
>> On Thu, Apr 9, 2020 at 11:57 PM Darren Duncan <[email protected]>
>> 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
>

view thread (19+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected]
  Subject: Re: Proposal: Drop support for Internet Explorer
  In-Reply-To: <CACCA4P3DYZJjx+_zNxhHfjzftN7no9QAvwWfyzCsBa+HjoMspg@mail.gmail.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox