Skip to content

Commit d8c55f6

Browse files
committed
Merge pull request #146 from aantron/html-deprecated
Mark some HTML attributes deprecated
2 parents 51e7b0c + 1b9d53e commit d8c55f6

File tree

2 files changed

+20
-4
lines changed

2 files changed

+20
-4
lines changed

lib/html_f.ml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -442,8 +442,6 @@ struct
442442

443443
let a_border = int_attrib "border"
444444

445-
let a_datapagesize = string_attrib "datapagesize"
446-
447445
let a_rules x =
448446
user_attrib C.string_of_big_variant "rules" x
449447

lib/html_sigs.mli

Lines changed: 20 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -564,8 +564,12 @@ module type T = sig
564564

565565
val a_align :
566566
[< | `Left | `Right | `Justify | `Char] wrap -> [> | `Align] attrib
567+
[@@ocaml.deprecated "Use CSS text-align"]
568+
(** @deprecated Use CSS text-align *)
567569

568570
val a_axis : cdata wrap -> [> | `Axis] attrib
571+
[@@ocaml.deprecated "Not supported in HTML5"]
572+
(** @deprecated Not supported in HTML5 *)
569573

570574
val a_colspan : number wrap -> [> | `Colspan] attrib
571575

@@ -575,15 +579,21 @@ module type T = sig
575579

576580
val a_scope :
577581
[< | `Row | `Col | `Rowgroup | `Colgroup] wrap -> [> | `Scope] attrib
582+
[@@ocaml.deprecated "Not supported in HTML5"]
583+
(** @deprecated Not supported in HTML5 *)
578584

579585
val a_summary : text wrap -> [> | `Summary] attrib
586+
[@@ocaml.deprecated "Move content elsewhere or to a <caption> child"]
587+
(** @deprecated Move content elsewhere or to a <caption> child *)
580588

581589
val a_border : pixels wrap -> [> | `Border] attrib
582-
583-
val a_datapagesize : cdata wrap -> [> | `Datapagesize] attrib
590+
[@@ocaml.deprecated "Use CSS border and/or border-width"]
591+
(** @deprecated Use CSS border and/or border-width *)
584592

585593
val a_rules :
586594
[< | `None | `Groups | `Rows | `Cols | `All] wrap -> [> | `Rules] attrib
595+
[@@ocaml.deprecated "Use CSS border"]
596+
(** @deprecated Use CSS border *)
587597

588598
val a_char : character wrap -> [> | `Char] attrib
589599
[@@ocaml.deprecated "The char attribute is not supported in HTML5"]
@@ -606,12 +616,20 @@ module type T = sig
606616
val a_data : Xml.uri wrap -> [> | `Data] attrib
607617

608618
val a_codetype : contenttype wrap -> [> | `Codetype] attrib
619+
[@@ocaml.deprecated "Not supported in HTML5"]
620+
(** @deprecated Not supported in HTML5 *)
609621

610622
val a_frameborder : [< | `Zero | `One] wrap -> [> | `Frameborder] attrib
623+
[@@ocaml.deprecated "Use CSS border"]
624+
(** @deprecated Use CSS border *)
611625

612626
val a_marginheight : pixels wrap -> [> | `Marginheight] attrib
627+
[@@ocaml.deprecated "Use CSS margin"]
628+
(** @deprecated Use CSS *)
613629

614630
val a_marginwidth : pixels wrap -> [> | `Marginwidth] attrib
631+
[@@ocaml.deprecated "Use CSS margin"]
632+
(** @deprecated Use CSS *)
615633

616634
val a_scrolling : [< | `Yes | `No | `Auto] wrap -> [> | `Scrolling] attrib
617635

0 commit comments

Comments
 (0)