Content deleted Content added
Drmccreedy (talk | contribs) →Replacing the codechart parameter: more needed |
|||
Line 117:
::I don't think we even need a new <code>codechartoverride</code> parameter, just retool the existing <code>codechart</code> parameter to handle <code>codechart=omit</code>, and you should be good to go. [[User:Vanisaac|Van]][[User talk:Vanisaac|Isaac]], GHTV<sup> [[Special:Contributions/Vanisaac|cont]]</sup><sub style="margin-left:-3.5ex"><small>[[WP:WPWR|WpWS]]</small></sub> 23:01, 30 July 2023 (UTC)
:::We would also need to change the behavior so if <code>codechart</code> is used, no link to the webpage/namelist is created. Initially, <code>codechart</code> was the only way to get the PDF link displayed then autogeneration was introduced. That's why I'm recommending changing the parameter name to <code>codechartoverride</code> to make it clear that it ''overrides'' the automatic generation of the PDF and webpage/namelist links. [[User:Drmccreedy|DRMcCreedy]] ([[User talk:Drmccreedy|talk]]) 23:41, 30 July 2023 (UTC)
::::The problem is that you're going to have to go through and change all the extant instances of <code>codechart</code> into <code>codechartoverride</code>, when all except the few "omit" instances are going to be absolutely identical in parameter value. And when looking at the functionality being implemented, <code>codechart=omit</code> is actually a better description of the behavior than <code>codechartoverride=omit</code>, while <code>codechart=http://example.com</code> is just as descriptive as <code>codechartoverride=http://example.com</code>. The only time an editor who would even notice a change in functionality is if they used the "omit" keyword. Changing the parameter name can only have a negative effect when editors who are used to the extant parameters don't get their desired results after the change. [[User:Vanisaac|Van]][[User talk:Vanisaac|Isaac]], GHTV<sup> [[Special:Contributions/Vanisaac|cont]]</sup><sub style="margin-left:-3.5ex"><small>[[WP:WPWR|WpWS]]</small></sub> 00:58, 31 July 2023 (UTC)
| |||