Opened on 12/02/2016 at 06:10:25 PM

Closed on 02/21/2018 at 05:11:58 PM

#4689 closed change (rejected)

Provide platform detection for Help Center website

Reported by: juliandoucette Assignee:
Priority: Unknown Milestone:
Module: Sitescripts Keywords:
Cc: matze, kvas, jsonesen, philll, saroyanm, greiner Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description (last modified by juliandoucette)

Background

  • The new Help Center website relies on platform detection to display platform specific directions
  • We have decided to include all platforms on a single page and show/hide platforms based on a classname injected into the HTML body element

What to change

  • adblockplus/cms or infrastructure must provide a means of implementing and testing platform detection for website development and deployment

Attachments (0)

Change History (7)

comment:1 Changed on 12/05/2016 at 08:04:03 PM by juliandoucette

  • Description modified (diff)

comment:2 Changed on 12/07/2016 at 01:35:16 PM by kvas

Do I understand it correctly that for the time being we're planning to go with client-side platform detection using JavaScript and that if we might need a server side solution it would be further in the future (1-2 years)?
If that is the case, I would say that by that time we (Operations) expect to have a solution for deploying static websites together with their related dynamic content in a way that all of this can be controlled by the website authors. This would make this issue not applicable in its current form, so perhaps we can close it.
Does this sound about right or does anyone have a different understanding?

comment:3 Changed on 12/07/2016 at 01:52:11 PM by juliandoucette

Do I understand it correctly that for the time being we're planning to go with client-side platform detection using JavaScript and that if we might need a server side solution it would be further in the future (1-2 years)?

The properties that we can use to detect the browser in JavaScript are depricated and may be removed by any major browser at any time.

We (Manvel and I) said that we are comfortable implementing a JavaScript solution temporarily if sitescripts/operations commits to developing a server side solution that will replace our JavaScript solution by the time we implement the next website that requires browser detection (probably in the next 1 or 2 quarters).

comment:4 Changed on 12/07/2016 at 01:56:47 PM by kvas

I see, so I got the timing wrong. Perhaps in this case it makes sense to keep this issue and rewrite it into a more generic way that's not tied to Help Center capturing the timeline as well. Would it be ok if I do this?

comment:5 Changed on 12/07/2016 at 01:57:36 PM by juliandoucette

Would it be ok if I do this?

Yes.

comment:6 Changed on 01/11/2017 at 06:14:04 PM by greiner

  • Cc greiner added

comment:7 Changed on 02/21/2018 at 05:11:58 PM by juliandoucette

  • Resolution set to rejected
  • Status changed from new to closed

On second thought, I don't think that this is necessary anymore. I think that it's very unlikely that we will not be able to implement proper detection on the client side or gracefully degrade in the few edge cases when we fail to detect browsers properly in the future.

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from (none).
 
Note: See TracTickets for help on using tickets.