Extend length of feature policy field

Created on 25 September 2019, almost 5 years ago

Updated 28 July 2023, 11 months ago

Problem/Motivation

When listing a larger number of feature policies to disable the text field does not offer enough space. The string "accelerometer 'none'; ambient-light-sensor 'none'; camera 'none'; document-domain 'none'; document-write 'none'; geolocation 'none'; gyroscope 'none'; legacy-image-formats 'none'; magnetometer 'none'; microphone 'none'; midi 'none'; payment 'none'; sync-xhr 'none'; usb 'none'; wake-lock 'none'; webauthn 'none'; vr 'none'" for example does not fit as it is wider than the current maximum of 255 characters.

Proposed resolution

Extend the length of the field to 2048 characters. This offers about 8x as much space and would be plenty for future extensions. Or no limit at all if that is acceptable. The feature policy spec seems to not list a maximum length.

A tiny patch is attached.

✨ Feature request

Status

Closed: duplicate

Version

2.0

Component

Code

Created by

🇳🇱Netherlands dennis_meuwissen

Live updates comments and jobs are added and updated live.

feature policy

Incomplete comments

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Comment about 1 year ago →
🇳🇱Netherlands ahmetburkan Utrecht
I agree with dennis_meuwissen → . This is a must with all the options that we have.
Status changed to Closed: duplicate 11 months ago8:01am 28 July 2023
Comment 11 months ago →
🇬🇧United Kingdom mcdruid 🇬🇧🇪🇺
It'd be better to tackle several admin form fields that could do with allowing longer inputs all at once.

We already did this in the D7 branch, and filed ✨ Text fields not big enough Fixed to consider forward-porting that change.

I'm going to mark this issue as a duplicate of that (and will copy credit across).

contrib.social Blog FAQ Discussions

Production build 0.69.0 2024