Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 501551

Summary: <dialog> no longer in html5.1
Product: [ECD] Orion Reporter: Carolyn MacLeod <Carolyn_MacLeod>
Component: JS ToolsAssignee: Curtis Windatt <curtis.windatt.public>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: curtis.windatt.public, Michael_Rennie, steve_northover
Version: unspecified   
Target Milestone: 15.0   
Hardware: PC   
OS: Windows 7   
Whiteboard:

Description Carolyn MacLeod CLA 2016-09-15 21:45:07 EDT
The dialog element has been removed from html 5.1 because there weren't enough browser implementations. It is still in html 5.2. Should we remove it from our html content assist for now? What is our policy regarding not-yet-supported elements?
Comment 1 Curtis Windatt CLA 2016-09-19 12:30:23 EDT
dialog does appear to be widely unsupported.  do you have any information on whether it will be dropped for 5.2 and forward?  We should likely remove it from our content assist.

when we do this we should start a wiki page explaining the changes made and why.  There is a doc and comments in the plugin explaining where the information was scraped from.
Comment 2 Carolyn MacLeod CLA 2016-09-19 13:23:17 EDT
All I have is this: https://w3c.github.io/html/changes.html#changes-fpwd
which (as of today) says,
"<dialog> element
This was removed from HTML 5.1 for lack of implementation, and may be removed from HTML 5.2 if not implemented."
Comment 3 Steve Northover CLA 2017-03-29 17:55:50 EDT
If this is easy to remove, just remove it.
Comment 5 Carolyn MacLeod CLA 2017-03-30 12:18:47 EDT
Do we also have attribute value completions for any of the other removed items at the link in comment 2? <menu type=toolbar> or <input type=datetime> etc?
(Note that MDN still has all of these, and their doc probably needs to be updated, but they [seem to try to] put an "experimental" icon/tag on them).

https://developer.mozilla.org/en-US/docs/Web/HTML/Element/dialog
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/menu
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/datetime
Comment 6 Curtis Windatt CLA 2017-03-30 12:59:36 EDT
(In reply to Carolyn MacLeod from comment #5)
> Do we also have attribute value completions for any of the other removed
> items at the link in comment 2? <menu type=toolbar> or <input type=datetime>
> etc?
> (Note that MDN still has all of these, and their doc probably needs to be
> updated, but they [seem to try to] put an "experimental" icon/tag on them).
> 
> https://developer.mozilla.org/en-US/docs/Web/HTML/Element/dialog
> https://developer.mozilla.org/en-US/docs/Web/HTML/Element/menu
> https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/datetime

No, we only have attr value completions for ARIA attrs. I don't see any further changes we need to make to the HTML tools.
Comment 7 Michael Rennie CLA 2017-03-30 13:45:02 EDT
(In reply to Curtis Windatt from comment #4)
> http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/
> ?id=50a992f9202f47e469ab80977990bb43e3ed1fab
> Fixed in master

Why comment it out rather than remove it (and the unused NLS string)?

Theres no value in leaving it there.
Comment 8 Curtis Windatt CLA 2017-03-30 14:02:27 EDT
(In reply to Michael Rennie from comment #7)
> Why comment it out rather than remove it (and the unused NLS string)?
> 
> Theres no value in leaving it there.

The data was scraped from MDN. If that process is run again it will put back the dialog tag.

I moved the comment to the top and removed the defunct NLS.

http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=f93b379fc9d5adf22c02fa985cea9d56219a588a