|
I see several options: I know we're not trying to offer full non-JS support, but going out of our way to deliberately break non-JS support seems stupid, so I really don't like a). Both b) and c) seem fine to me. Calling this a decision required is disrespectful to real decisions we need to make. I made it work correctly: I'm fine with making this page work, but the lack of a guideline still stands, and will inevitably come up again in the future. Fair enough. I'll add it back to the meeting topic then – my position, for the record, is basically reduced to "bots should be able to edit anything on MB" – so, javascript is fine for editor niceties like "when you aren't using a general-purpose programming language it's hard to add fields to a page" but not for basic rendering – unless there's an alternative interface. So, the relationship editor is fine because everything that can be done there can be done other ways, if a bot wants to. The release editor may not be fine by my metric (not sure about tracklist editing automation, or the recordings tab), but that's only one of many problems with it. This page, pre-pull-37, isn't okay because loading the page without JS then submitting without making changes, changes things; post-pull-37, javascript is only used for the add/remove field functionality a script written in a general programming language won't need. |
||||||||||||||||||||||||||||||||||||||||||||||||||
Needs a decision - do we deny this page to work, or do we make the page work correctly?