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

update html 5 package #565

Closed
GoogleCodeExporter opened this issue Mar 25, 2015 · 18 comments
Closed

update html 5 package #565

GoogleCodeExporter opened this issue Mar 25, 2015 · 18 comments

Comments

@GoogleCodeExporter
Copy link

The html 5 package should be reviewed and updated.

Examples

missing input types:
- tel
- color
missing input attributes
- placeholder

missing video and audio attributes
- controls
- autoplay

Original issue reported on code.google.com by philippe...@gmail.com on 17 Apr 2010 at 8:52

@GoogleCodeExporter
Copy link
Author

Original comment by obi...@gmail.com on 1 Jun 2010 at 11:02

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Updated the HTML5 package according to: 
http://dev.w3.org/html5/html4-differences/
 - added the following tags: hgroup, figcaption, bdi, wbr, embed, output
 - updated the marked attribute (m -> mark)

Issues I've seen:
1) There's some confusion about the location of specialized event handlers. For 
example: the WASearchInputTag has a #onSearch: method which adds an event 
handler specialized for that tag, while WATagBrush has an onInput methods which 
belongs only to inputs that support it. Maybe this needs some cleanup.
2) The idea of using an HTML5 package on top of the HTML4 package doesn't seem 
right. Some elements have been removed from HTML5, and new attributes were 
added to elements that already exist in HTML4. Is there a need to enforce these 
rules, or should we simply ignore this, and let the user develop non-compliant 
code (either for html4 or html5)?
3) The following attributes are in the Seaside-Canvas-Tags, while looks like 
they belong to HTML5: keygen, ruby, rt, rp


Name: Seaside-HTML5-as.12
Author: as
Time: 1 February 2011, 6:14:52 pm
UUID: bce2bd6c-1285-004a-a9ab-78a94781dece
Ancestors: Seaside-HTML5-lr.11

 - implemented hgroup, figcaption, bdi, wbr, embed, output
 - updated marked element

Name: Seaside-Tests-HTML5-as.18
Author: as
Time: 1 February 2011, 6:15:57 pm
UUID: b756bfce-add7-e84c-90df-30e988f32ecc
Ancestors: Seaside-Tests-HTML5-pmm.17

updated test for the marked element

Original comment by avish...@gmail.com on 1 Feb 2011 at 4:16

  • Added labels: ****
  • Removed labels: ****

Attachments:

@GoogleCodeExporter
Copy link
Author

re: #2, we've never tried to enforce DTD-correct HTML (well, we did actually, 
in Seaside 0.9, but it was a nightmare), satisfying ourselves with generating 
structurally correct XHTML. One reason for this is that developers often need 
to generate non-compliant HTML in order to make things work properly in 
different browsers, but it's also just a big ugly problem that I don't think we 
need to be too concerned about.

Original comment by jfitz...@gmail.com on 1 Feb 2011 at 11:48

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

I agree with Julian. At one point we'll probably have to merge the HTML5 
package into the html 4 canvas.

Original comment by philippe...@gmail.com on 2 Feb 2011 at 6:38

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Thanks, I merged it. I also "seasidified" some of the name. In general:
  * we use #url: instead of src/href/...
  * we try to avoid abbreviations

You're right about the ruby and keygen elements, I moved them to HTML5.

Original comment by philippe...@gmail.com on 2 Feb 2011 at 7:02

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Thanks for the constructive criticism, it really helps and gets me quickly into 
things.

If there is no need for enforcing correct HTML, then it really is a good idea 
to merge the packages.

Original comment by avish...@gmail.com on 2 Feb 2011 at 7:58

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

The following are all included already.

input types:
- tel
- color
input attributes
- placeholder

will have to check audio and video.

Original comment by philippe...@gmail.com on 8 Feb 2011 at 6:23

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Added required audio and video attributes, merged them as a media element 
(according to spec), and added functional tests accordingly.


Name: Seaside-Tests-HTML5-as.21
Author: as
Time: 8 February 2011, 3:37:19 pm
UUID: 5145c451-e047-6544-8e73-2348ec6174ef
Ancestors: Seaside-Tests-HTML5-pmm.20

Added audio test, merged video test into WAHtml5MediaElementsTest

Name: Seaside-HTML5-as.18
Author: as
Time: 8 February 2011, 3:39:39 pm
UUID: c94550c3-4167-9342-b2f5-803f461488d1
Ancestors: Seaside-HTML5-pmm.17

 - Merged audio and video tags under WAMediaElementTag (according to spec).
 - Added autoplay, controls, loop, preload and url attributes.
 - Added the poster attribute to the video tag.

Original comment by avish...@gmail.com on 8 Feb 2011 at 1:37

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

According to http://dev.w3.org/html5/html4-differences/ the HTML5 package 
contains all new functionality and tags.

What's missing:
 - inclusion of new attributes http://dev.w3.org/html5/html4-differences/#new-attributes
 - changed elements http://dev.w3.org/html5/html4-differences/#changed-elements
 - changed attributes http://dev.w3.org/html5/html4-differences/#changed-attributes

Original comment by avish...@gmail.com on 8 Feb 2011 at 1:41

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

I went through the list and this is what I believe is still not yet done:

- area is missing and so is map (should probably be a different issue)
- value on li is missing (we have no dedicated li tag class)
- form attribute for input, output, select, textarea, button and fieldset
- scoped attribute on style and style as flow content
- async attribute on style
- manifest attribute on html
- sizes attribute on link
- sandbox, seamless, and srcdoc attriutes on iframe

Name: Seaside-HTML5-pmm.19
Author: pmm
Time: 8 February 2011, 10:17:04 pm
UUID: b0de019c-bfae-424d-9ac4-fb54fecfaca5
Ancestors: Seaside-HTML5-as.18

- Issue 565:    update html 5 package
- added some missing attributes on ol
- http://code.google.com/p/seaside/issues/detail?id=565
- the following global attributes contenteditable, contextmenu, data-*, 
draggable, requirement, hidden, role, aria-*, spellcheck

Original comment by philippe...@gmail.com on 8 Feb 2011 at 9:17

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

The HTML5 spec is still changing and it's hard to keep up with the latest 
changes.  Since I'm developing a Seaside course, I decided to go through the 
current HTML5 spec (Editor's Draft 23 September 2011) and compare what element 
tags are available in that spec versus the tags we support in Seaside (Seaside 
3.0 - 8 released with VW7.8).  In general, Seaside has quite good support for 
HTML5.  Here are the differences I've found:

 Note:  I'm simply offering this list as information for anyone interested.  I needed to research this for the course and thought it would be good to share the results.

 The following tags are inconsistent between Seaside and the current version of HTML5
   mark   (already reported)

 Tags in HTML5 not supported by Seaside (some of these are poor style and should use newer tags but are still supported in HTML):
   hgroup
   figcaption
   s
   i (should use emphasis)
   b (should use strong)
   u (shouldn't really use at all)
   bdi
   bdo
   wbr
   embed
   audio (interestingly, a WAAudioTag class exists with no call from the canvas)
   track
   map (WAImageMapTag class uses an anchor tag)
   area
   output
   summary

 The following tags are supported by Seaside but aren't in the current HTML5 spec (no problem keeping them for backward compatibility but perhaps they should be marked as deprecated):
   rubyBase
   rubyBaseContainer
   rubyTextContainer
   datagrid
   acronym
   dialog
   teletype

 David Buck

Original comment by philippe...@gmail.com on 26 Sep 2011 at 11:54

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

> Tags in HTML5 not supported by Seaside (some of these are poor style and
 > should use newer tags but are still supported in HTML):
 >   hgroup

#headingGroup there but missing tests

 >   figcaption

 missing

 >   s
 >   i (should use emphasis)
 >   b (should use strong)
 >   u (shouldn't really use at all)

All of these are deprecated in HTML 4 in favor of CSS and intentionally missing.

 >   bdi

 #bidirectional, there but missing tests, should probably be #biDirectional. (Julian disagrees later)

 >   bdo

 missing indeed

 >   wbr

 #lineBreakOpportunity, there but missing tests

 >   embed

 #embed there but missing tests

 >   audio (interestingly, a WAAudioTag class exists with no call from the
 > canvas)

#audio, even has tests

 >   track

 missing indeed

 >   map (WAImageMapTag class uses an anchor tag)

missing indeed

 >   area

 missing indeed

 >   output

 We have a WAOutputTag but it's never used :-(

 >   summary

 missing indeed

 > The following tags are supported by Seaside but aren't in the current HTML5
 > spec (no problem keeping them for backward compatibility but perhaps they
 > should be marked as deprecated):
 >   rubyBase
 >   rubyBaseContainer
 >   rubyTextContainer

Indeed, they seem to be replaced by rt and rp

 >   datagrid

 funny, yes missing

 >   acronym

 "deprecated" but still in HTML 4, therefore likely to stay

 >   dialog

 Indeed

 >   teletype

 "deprecated" but still in HTML 4, therefore likely to stay

 Cheers
Philippe

Original comment by philippe...@gmail.com on 26 Sep 2011 at 11:54

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Name: Seaside-HTML5-pmm.25
Author: pmm
Time: 26 September 2011, 6:41:56 pm
UUID: b5854834-9749-4ff7-960d-919e9c7c11fb
Ancestors: Seaside-HTML5-pmm.24

- Issue 565:    update html 5 package

Name: Seaside-Tests-HTML5-pmm.28
Author: pmm
Time: 26 September 2011, 6:42:37 pm
UUID: bc2097f1-6b54-4ef1-9aca-66a066ada6b4
Ancestors: Seaside-Tests-HTML5-jok.27

- Issue 565:    update html 5 package

Original comment by philippe...@gmail.com on 26 Sep 2011 at 4:43

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

The following elements are still missing

dbo (bidirectionalOverride)
track
map
area
output

Original comment by philippe...@gmail.com on 26 Sep 2011 at 4:43

  • Changed state: Started
  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Name: Seaside-HTML5-pmm.26
Author: pmm
Time: 28 September 2011, 7:12:27 pm
UUID: 5ddb0b06-c5c3-4b07-a8e4-047ef273930f
Ancestors: Seaside-HTML5-pmm.25

- add <output>

Name: Seaside-Tests-HTML5-pmm.30
Author: pmm
Time: 28 September 2011, 7:13:18 pm
UUID: 69720a2c-a5b2-4876-a23c-567cecc78de7
Ancestors: Seaside-Tests-HTML5-pmm.29

- add <output>

Original comment by philippe...@gmail.com on 28 Sep 2011 at 5:13

  • Added labels: Version-Seaside3.1
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Name: Seaside-HTML5-pmm.27
Author: pmm
Time: 28 September 2011, 7:39:18 pm
UUID: 54921f43-a0f7-45f6-95f6-ea286cb2e1da
Ancestors: Seaside-HTML5-pmm.26

- add <track>

Name: Seaside-Tests-HTML5-pmm.31
Author: pmm
Time: 28 September 2011, 7:39:51 pm
UUID: 49753448-fdd3-478a-97f1-81b820ea73f5
Ancestors: Seaside-Tests-HTML5-pmm.30

- add <track>

Original comment by philippe...@gmail.com on 28 Sep 2011 at 5:40

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Name: Seaside-HTML5-pmm.28
Author: pmm
Time: 28 September 2011, 8:03:09 pm
UUID: 4c92e8bc-0aa7-4bb5-99f5-5f77d429a0cd
Ancestors: Seaside-HTML5-pmm.27

- add <bdo>

Name: Seaside-Tests-HTML5-pmm.32
Author: pmm
Time: 28 September 2011, 8:03:51 pm
UUID: 918e5e97-b176-42b8-b47b-2cbf2a88324c
Ancestors: Seaside-Tests-HTML5-pmm.31

- add <bdo>

Original comment by philippe...@gmail.com on 28 Sep 2011 at 6:04

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Work has been done on the area tag. See 
http://smalltalkhub.com/#!/~a3aan/SeasideSprintArea. Also see issue 
https://code.google.com/p/seaside/issues/detail?id=826.

Original comment by adriaan....@gmail.com on 24 Aug 2014 at 10:41

  • Added labels: ****
  • Removed labels: ****

@jbrichau jbrichau closed this as completed Nov 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants