[GitHub] [daffodil-vscode] tuxji commented on pull request #286: renamed wholeLine variable to triggerText

[GitBox <gi...@apache.org>]

tuxji commented on PR #286:
URL: https://github.com/apache/daffodil-vscode/pull/286#issuecomment-1242132731

   > In this example, I think we want to autocomplete with the first namespace prefix that matches the XML schema namespace, i.e. no prefix
   
   I agree we want no name prefix, but I want a smarter heuristic than first prefix.  I usually put my schema element's attributes on separate lines (one attribute per line) and sort these lines in my editor for consistency, which puts the no name prefix on the last line:
   
   ```xml
   <schema
     targetNamespace="http://example.com"
     xmlns:dfdl="http://www.ogf.org/dfdl/dfdl-1.0/"
     xmlns:ex="http://example.com"
     xmlns:xs="http://www.w3.org/2001/XMLSchema"
     xmlns="http://www.w3.org/2001/XMLSchema">
   ```
   
   In the case when a schema has multiple prefixes for the same namespace ("", "xs", and "xsd"), I would prefer that the shorter prefix supersede the longer prefix regardless of the order they're defined in, but I can live with text lexical order "wins".


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@daffodil.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org