Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(164)

Unified Diff: third_party/WebKit/LayoutTests/fast/forms/fieldset/fieldset-display-flex.html

Issue 2150003005: Add grid/flex layout support for <fieldset> (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: updated TestExpectations and fixed legend-after-margin-vertical-writing-mode.html Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/fast/forms/fieldset/fieldset-display-flex.html
diff --git a/third_party/WebKit/LayoutTests/fast/forms/fieldset/fieldset-display-flex.html b/third_party/WebKit/LayoutTests/fast/forms/fieldset/fieldset-display-flex.html
new file mode 100644
index 0000000000000000000000000000000000000000..4947ef5ad84254d8007790ef7a36a176978b1c8c
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/fast/forms/fieldset/fieldset-display-flex.html
@@ -0,0 +1,20 @@
+<!DOCTYPE html>
cbiesinger 2016/07/21 19:54:16 You should keep this line; we don't usually want t
Gleb Lanbin 2016/07/21 20:49:01 Done.
+<html>
+<head>
Manuel Rego 2016/07/20 06:00:23 You don't need html, head and body tags on this te
Gleb Lanbin 2016/07/21 18:25:30 Done.
+<title>Test Case with FIELDSET as flex container</title>
Manuel Rego 2016/07/20 06:00:23 We don't usually use the title to describe the tes
Gleb Lanbin 2016/07/21 18:25:31 Done.
+<style>
+* {
+ padding: 1em;
Manuel Rego 2016/07/20 06:00:23 This is not needed.
Gleb Lanbin 2016/07/21 18:25:31 Done.
+}
+</style>
+</head>
+
Manuel Rego 2016/07/20 06:00:23 It's usually nice to add a paragraph explaining wh
Gleb Lanbin 2016/07/21 18:25:31 Done.
+<body>
+<fieldset style="display: flex;">
+ <legend>Fieldset with display: flex</legend>
+ <div>these fields</div>
+ <div>shouldn't be</div>
+ <div>stacked vertically</div>
+</fieldset>
+</body>
+</html>
Manuel Rego 2016/07/20 06:00:23 And last thing about the test, I don't see any goo
Gleb Lanbin 2016/07/21 18:25:31 Done.

Powered by Google App Engine
This is Rietveld 408576698