Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

list of todos #36

Closed
22 of 25 tasks
CHAOSdonut opened this issue Dec 12, 2022 · 7 comments
Closed
22 of 25 tasks

list of todos #36

CHAOSdonut opened this issue Dec 12, 2022 · 7 comments
Labels
enhancement New feature or request

Comments

@CHAOSdonut
Copy link
Collaborator

CHAOSdonut commented Dec 12, 2022

profiles/logius.js:

  • logius/fix-md-elements.js: check
  • w3c/level.js: check if this must be skipped
  • w3c/group.js: check if this must be skipped

scr/logius:

fix-md-elements.js

  • check if algorithm is correct!
  • do nothing if this config is not set?

headers.js

  • strip w3c specific content
  • document license types for Geonovum
  • check fixed, static urls
  • no clear 'ED' status in this version
  • check path generation
  • should the prev-/spectype always be in the WP URL too? #41
  • add an extra check, because now it seems that showPreviousVersion is true in (too) many cases?
  • check GN STATUSES in templates
  • check if logius will still use
  • Pieter: do we need these?
  • Pieter: is the comment below valid in this version?
  • Pieter valid?
  • pieter commented out
  • revert geonvum alterations

splitmarkdownheaders.js

  • refactor + correct commenting etc
  • testing
  • Todo

src/styles

dfn-panel.css.js

tests/test-main.js

  • Remove this when browsers add support for top level await

test/testindexStable.html

  • when becomes a implemented, we should really use that.
@CHAOSdonut CHAOSdonut added bug Something isn't working enhancement New feature or request and removed bug Something isn't working labels Dec 12, 2022
@CHAOSdonut
Copy link
Collaborator Author

CHAOSdonut commented Dec 12, 2022

#38
check if algorithm is correct! / do nothing if this config is not set?
if-statement opgeschoond en gekeken of de functie werkt

@CHAOSdonut
Copy link
Collaborator Author

CHAOSdonut commented Dec 12, 2022

#39
W3C/level & W3C/group
Dit zijn functionaliteiten van W3C, ik verwacht niet dat gebruikers van dit profiel het vaak nodig zullen hebben maar het verwijderen van de functionaliteit heeft geen meerwaarden.

@CHAOSdonut
Copy link
Collaborator Author

tests/test-main.js

Remove this when browsers add support for top level await
alle moderne versies van browsers (behalve IE) supporten top level await: https://caniuse.com/?search=top%20level%20await

deze todo komt van W3C dus ik laat hem wel staan

@CHAOSdonut
Copy link
Collaborator Author

todos die bij W3C vandaan komen moeten blijven staan. Als we ze toch willen sluiten is het beter om een pull request in te dienen bij W3C

@CHAOSdonut
Copy link
Collaborator Author

CHAOSdonut commented Dec 14, 2022

test/testindexStable.html

when <dialog> becomes a implemented, we should really use that.

het is een W3C todo maar hij laat de todo comment achter in de snapshot.html, dit is niet echt netjes. Het is me niet helemaal duidelijk wat er bedoelt word met: "<dialog> becomes a implemented" is het een gramaticale fout of is implemented een bepaalde type/class.

@sanderke
Copy link
Member

todos die bij W3C vandaan komen moeten blijven staan. Als we ze toch willen sluiten is het beter om een pull request in te dienen bij W3C

Als het in het Logius-profiel staat komt het waarschijnlijk doordat begonnen is met een kopie van een W3C-bestand en zijn die todo's irrelevant en te verwijderen uit de kopie.

@CHAOSdonut
Copy link
Collaborator Author

todos die bij W3C vandaan komen moeten blijven staan. Als we ze toch willen sluiten is het beter om een pull request in te dienen bij W3C

Als het in het Logius-profiel staat komt het waarschijnlijk doordat begonnen is met een kopie van een W3C-bestand en zijn die todo's irrelevant en te verwijderen uit de kopie.

Ook als de todos ook bij de huidige versie van W3C nog aanwezig zijn?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants