[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=55383

Mark Thomas  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|NEW |RESOLVED

--- Comment #56 from Mark Thomas  ---
This was completed for 8.5.x onwards. It had been left open as it had not been
addressed for 7.0.x. Now 7.0.x is EOL (without this issue being addressed) I am
marking this issue as resolved.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #55 from Tristan  ---
There's definitely some cross-browser isuses going on here. Chrome 46 and IE
10,11 have odd issues.

https://www.learnthreejs.com/load-3d-model-using-three-js-obj-loader/

Can anybody confirm with the resource above? Thanks.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=55383

Michael Osipov <1983-01...@gmx.net> changed:

   What|Removed |Added

 CC||1983-01...@gmx.net

--- Comment #54 from Michael Osipov <1983-01...@gmx.net> ---
Hi Konstantin,

tremendous improvement with a lot of effort put it. Thank you very much for
that!

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=55383

manoripriyadarshani  changed:

   What|Removed |Added

 CC||kampriyadarshani326@gmail.c
   ||om

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

Alex Bretet bre...@gmail.com changed:

   What|Removed |Added

 CC||bre...@gmail.com

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #53 from Konstantin Preißer kpreis...@apache.org ---
Hi Konstantin,
thank you for the review.

(In reply to Konstantin Kolinko from comment #52)
 (In reply to Konstantin Preißer from comment #49)
  
  [1] http://www.google.com/fonts/specimen/Open+Sans
 
 The *.woff that you committed in r1546663 - where do they come from?
 
 I am worried whether we need to add some attribution if we are
 redistributing them,
 and how do we get updates for them if there are any.

I took the .woff files by going to [3], selecting the needed fonts, and then
opening the generated CSS link [4].

This is a Google-generated CSS that contains links to .ttf, .woff and other
formats depending on the browser. I then downloaded the .woff ones.

If there's a problem with taking these .woff files directly, I think it should
be possible to use some tool to convert the .ttf to .woff files (but then one
needs to take care not include only specific characters of the font, to reduce
the file size).

 Thus far I have two trails
 a) Google Code repository for Google Fonts project, [2].
 
 There are *.ttf files.
 
 b) I went to [1] clicked on Open Open Sans in Google Fonts which navigated
 to [3].
 
 At [3] I click download button (a small down arrow button at the
 top-right part of the page). It downloads a 1.2Mb zip that contains 10 *.ttf
 files and LICENSE.txt (a copy of Apache License 2.0).
 
 [2]
 https://code.google.com/p/googlefontdirectory/source/browse/apache/opensans/
 [3] https://www.google.com/fonts#UsePlace:use/Collection:Open+Sans
 
 So thus far
 1. It is clear that the files are covered by Apache License 2.0
 2. There is no NOTICE file
 3. These are *.ttf files, not *.woff ones.

[4]
http://fonts.googleapis.com/css?family=Open+Sans:400,400italic,600,600italic,700,700italic

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #52 from Konstantin Kolinko knst.koli...@gmail.com ---
(In reply to Konstantin Preißer from comment #49)
 
 [1] http://www.google.com/fonts/specimen/Open+Sans

The *.woff that you committed in r1546663 - where do they come from?

I am worried whether we need to add some attribution if we are redistributing
them,
and how do we get updates for them if there are any.

Thus far I have two trails
a) Google Code repository for Google Fonts project, [2].

There are *.ttf files.

b) I went to [1] clicked on Open Open Sans in Google Fonts which navigated to
[3].

At [3] I click download button (a small down arrow button at the top-right
part of the page). It downloads a 1.2Mb zip that contains 10 *.ttf files and
LICENSE.txt (a copy of Apache License 2.0).

[2]
https://code.google.com/p/googlefontdirectory/source/browse/apache/opensans/
[3] https://www.google.com/fonts#UsePlace:use/Collection:Open+Sans

So thus far
1. It is clear that the files are covered by Apache License 2.0
2. There is no NOTICE file
3. These are *.ttf files, not *.woff ones.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #49 from Konstantin Preißer kpreis...@apache.org ---
Hi,

I'd like to switch the font used for the Tomcat Website and Docs (currently
Lucida Sans Unicode) to Open Sans [1] which is released by Google using a
Apache 2.0 Licence. This also means that the font can be displayed on every
platform like Windows, Mac and Linux.

I plan to add the Font files (.woff) (the variant which contains only the Latin
characters) directly instead of using a CSS link to Google, so that the Site
and the Docs (which may be viewed offline) don't depend on external Google
resources.

Is it OK to commit?


[1] http://www.google.com/fonts/specimen/Open+Sans

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #50 from Christopher Schultz ch...@christopherschultz.net ---
(In reply to Konstantin Preißer from comment #49)
 I'd like to switch the font used for the Tomcat Website and Docs (currently
 Lucida Sans Unicode) to Open Sans [1] which is released by Google using
 a Apache 2.0 Licence. This also means that the font can be displayed on
 every platform like Windows, Mac and Linux.

It's worth noting that the current CSS for font-family is just a suggestion to
the browser. The font you get depends very heavily on the environment where the
pages are being viewed.

 I plan to add the Font files (.woff) (the variant which contains only the
 Latin characters) directly instead of using a CSS link to Google, so that
 the Site and the Docs (which may be viewed offline) don't depend on external
 Google resources.

I'm not a huge fan of WOFF for a few reasons:

1. It requires yet another file to download, and font files tend to be big (I
couldn't find a download link so I don't know the exact size of the Open Sans
font).

2. Most browsers I've seen tend to render the page as quickly as possible, and
then apply the custom-font afterward, which results in a weird post-adjustment
of the font, even for fonts that are already downloaded. I haven't done
extensive testing recently, so it's possible that this is no longer an issue.

3. Ultimately, the font choice is up to the browser, anyway, so you can't
really force them to use Open Sans.

Given the above, why bother with a change at all? I know designers get all hot
and bothered by things like kerning, serif angles, etc. but I'm not sure that
adding another font-choice here really adds anything.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #51 from Konstantin Preißer kpreis...@apache.org ---
Hi Christopher, thanks for your comments.

(In reply to Christopher Schultz from comment #50)
 (In reply to Konstantin Preißer from comment #49)
  I'd like to switch the font used for the Tomcat Website and Docs (currently
  Lucida Sans Unicode) to Open Sans [1] which is released by Google using
  a Apache 2.0 Licence. This also means that the font can be displayed on
  every platform like Windows, Mac and Linux.
 
 It's worth noting that the current CSS for font-family is just a suggestion
 to the browser. The font you get depends very heavily on the environment
 where the pages are being viewed.
 
  I plan to add the Font files (.woff) (the variant which contains only the
  Latin characters) directly instead of using a CSS link to Google, so that
  the Site and the Docs (which may be viewed offline) don't depend on external
  Google resources.
 
 I'm not a huge fan of WOFF for a few reasons:
 
 1. It requires yet another file to download, and font files tend to be big
 (I couldn't find a download link so I don't know the exact size of the Open
 Sans font).

Google Fonts allows to explicitely select the charsets which should be included
in the Font, to keep the file sizes down. E.g. in this case it would be (400,
400 italics, 600, 600 italics): 84 KB, which should be an acceptable size in
today's relations (e.g. the ASF feather PNG image (asf-feather.png) has 40 KB).

 2. Most browsers I've seen tend to render the page as quickly as possible,
 and then apply the custom-font afterward, which results in a weird
 post-adjustment of the font, even for fonts that are already downloaded. I
 haven't done extensive testing recently, so it's possible that this is no
 longer an issue.

This applies if you visit the page the first time: Some browsers (Firefox) will
display the page with an alternate font until the WOFF is downloaded, then
switch to the WOFF. Other browsers (Chrome) will display spaces until the WOFF
is loaded.

However, after that if you browse on other pages then the WOFF is already in
the cache and the browser will use it immediately (at least this is the case
with Firefox, IE and Chrome), so there should be no issues when browsing
through various links on the site.

 3. Ultimately, the font choice is up to the browser, anyway, so you can't
 really force them to use Open Sans.

Yes (but this should also be the case with local fonts).

 
 Given the above, why bother with a change at all? I know designers get all
 hot and bothered by things like kerning, serif angles, etc. but I'm not sure
 that adding another font-choice here really adds anything.

I'm also not a designer but I think it looks a bit more modern than the
currently used Lucida Sans Console (and as it is a platform-independent format,
other platforms will be able to use the font).

I use it on my own websites and e.g. this site uses it:
http://www.html5rocks.com/en/tutorials/websockets/basics/

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #48 from Konstantin Preißer kpreis...@apache.org ---
Improvements of design and markup have been applied to the Tomcat Native
documentation (to match Tomcat 8 documentation) and will be included in
TC-Native 1.1.30 onwards.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #44 from Konstantin Preißer prei...@web.de ---
Created attachment 30862
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=30862action=edit
Use local ASF image

Hi,

previously, for the Tomcat docs I used this URL for the Apache Logo (an
alphatransparent PNG): http://www.apache.org/images/feather.png
However, as the Tomcat Docs are included in the release package, I think it
should use a local image instead.

The image added by the patch (webapps/docs/images/asf-feather.png) is the one
from the above URL.

The patch also removes the line 'Loading comments…' as it is not removed by the
Comments script.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #45 from Konstantin Preißer prei...@web.de ---
Created attachment 30863
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=30863action=edit
Improve markup for Welcome pages

This patch improves the HTML markup in the Welcome pages (welcome.main.html and
welcome.bin.html).

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #46 from Konstantin Preißer prei...@web.de ---
Created attachment 30864
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=30864action=edit
Improve markup for Tomcat Examples - Part 1

Hi,

this patch improves the HTML markup in the Tomcat Examples webapp (currently,
only the index pages and the websocket examples).

Note: For the WebSocket examples, I switched from HTML to XHTML because every
browser that supports WebSockets (e.g. IE = 10) also supports XHTML (e.g. IE
= 9), and because the Snake Example contained an XHTML doctype (but was not
fully XHTML-compatible).

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #47 from Mark Thomas ma...@apache.org ---
Patches all applied. Thanks.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #43 from Mark Thomas ma...@apache.org ---
(In reply to Konstantin Preißer from comment #42)
 Created attachment 30824 [details]

 this patch improves the markup in the Tomcat Site XMLs. After applying, the
 HTMls should validate against a HTML5 validator.

Patch applied. Thanks again.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #42 from Konstantin Preißer prei...@web.de ---
Created attachment 30824
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=30824action=edit
Updated Tomcat Site XMLs

Hi,

this patch improves the markup in the Tomcat Site XMLs. After applying, the
HTMls should validate against a HTML5 validator.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #33 from Violeta Georgieva violet...@apache.org ---
Created attachment 30812
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=30812action=edit
Missing background color IE9 top

The background color on the top is missing when using IE9. See the attached
screenshot.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #34 from Konstantin Kolinko knst.koli...@gmail.com ---
Created attachment 30813
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=30813action=edit
Broken layout in IE8

In IE8 (WinXP) the background color is missing as well.

Plus to that the top of the page is broken.
The Tomcat cat and Apache Feather images are located above the page title,
instead of floating at the left/right sides of it.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #35 from Konstantin Preißer prei...@web.de ---
Hi,
(In reply to Violeta Georgieva from comment #33)
 The background color on the top is missing when using IE9. See the attached
 screenshot.

I think this is because background: linear-gradient(...) CSS is first
supported by IE 10.

(In reply to Konstantin Kolinko from comment #34)
 Created attachment 30813 [details]
 Broken layout in IE8
 
 In IE8 (WinXP) the background color is missing as well.
 
 Plus to that the top of the page is broken.
 The Tomcat cat and Apache Feather images are located above the page title,
 instead of floating at the left/right sides of it.

I think this is because I used new HTML5 elements like header id=header
which have a DOM in IE8 as header id=header/header.../header//header,
so the CSS doesn't apply to them. One would have to use something like
headerdiv id=header.../div/header for it to work in IE8.

I hoped it wouldn't be necessary, but I will look into adding workarounds for
old IEs. :)

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #36 from Konstantin Preißer prei...@web.de ---
Created attachment 30814
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=30814action=edit
Adds work-arounds for IE ≤ 9.

This patch adds work-arounds for IE ≤ 9 to fix the layout on these IE versions.

I used conditional comments to include a IE fix CSS file, and to use div
id=... for IE ≤ 9, while IE ≥ 10 and other browsers will use header
id=... as IE 10 dropped support for conditional comments in Full Standards
Mode.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #37 from Konstantin Preißer prei...@web.de ---
Created attachment 30815
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=30815action=edit
Adds work-arounds to Tomcat 8 Docs for IE ≤ 9.

This one adds the same work-arounds to the Tomcat 8 docs.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #38 from Mark Thomas ma...@apache.org ---
Applied. Many thanks.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

Konstantin Preißer prei...@web.de changed:

   What|Removed |Added

  Attachment #30814|0   |1
is obsolete||

--- Comment #39 from Konstantin Preißer prei...@web.de ---
Created attachment 30818
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=30818action=edit
Layout fixes for old browsers without conditional comments

After Mark applied the patch, it turned out that the conditional comments for
the IE fixes broke the CGI scripts on the download pages (see [1]). I'm sorry
for that.

Therefore, this patch adds layout fixes for IE = 8 and and old Firefoxes (3.x)
without using conditional comments.


[1] http://markmail.org/message/rya7c6iu6vvbdkmw

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

Konstantin Preißer prei...@web.de changed:

   What|Removed |Added

  Attachment #30815|0   |1
is obsolete||

--- Comment #40 from Konstantin Preißer prei...@web.de ---
Created attachment 30819
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=30819action=edit
Doc layout fixes for old browsers without conditional comments

This one will apply the changes to the Tomcat 8 docs.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #41 from Mark Thomas ma...@apache.org ---
Both applied. Thanks.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #31 from Konstantin Preißer prei...@web.de ---
Created attachment 30809
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=30809action=edit
Ppatch for improving HTML markup  style in the Tomcat Site

Hi,

here is a patch for the main Tomcat Site to improve the HTML markup and the
layout, similar to the one for Tomcat 8 Doc.

The added image (xdocs/images/tomcat.png) is the same one as in attachment
30740.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #32 from Mark Thomas ma...@apache.org ---
(In reply to Konstantin Preißer from comment #31)
 Created attachment 30809 [details]
 Patch for improving HTML markup  style in the Tomcat Site

Patch applied. Waiting for the reaction on the users mailing list...

I haven't enjoyed committing a patch so much in ages. Thank you. It is a huge
improvement. But don't let that stop you trying to make it even better ;)

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #30 from Mark Thomas ma...@apache.org ---
(In reply to Konstantin Preißer from comment #29)
 With this patch, the improvement of HTML markup in the Tomcat 8 docs should
 be complete.

Applied. Many, many thanks.

 I think I can look into applying the new markup/design for the main Tomcat
 Site now.

Fantastic. Go for it.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #28 from Mark Thomas ma...@apache.org ---
(In reply to Konstantin Preißer from comment #27)
 Created attachment 30793 [details]
 Part 3 of updated documentation XMLs

Patch applied. Many thanks.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #29 from Konstantin Preißer prei...@web.de ---
Created attachment 30797
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=30797action=edit
Part 4 of updated documentation XMLs

Hi,

here is the final part of the documentation update.
With this patch, the improvement of HTML markup in the Tomcat 8 docs should be
complete.

Note: In the doc about the DefaultServlet [1], I also modified a dead link to
Servlet 2.3 to point to http://jcp.org/en/jsr/detail?id=340 (Servlet 3.1).


(In reply to Mark Thomas from comment #10)
 My current view is that once we are happy with this for 8.0.x (I'm pretty
 much there - I can't speak for the other committers) we apply similar
 changes to 7.0.x, 6.0.x and the main Tomcat site (all are built with
 essentially the same mechanism).

I think I can look into applying the new markup/design for the main Tomcat Site
now.


[1] http://tomcat.apache.org/tomcat-8.0-doc/funcspecs/fs-default.html

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #27 from Konstantin Preißer prei...@web.de ---
Created attachment 30793
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=30793action=edit
Part 3 of updated documentation XMLs

Hi,

here is another part of updated Documentation XMLs.

I modified the CSS so that a table with class=detail-table looks the same as
class=defaultTable. If you don't like it, you (or I) can change it so that a
detail-table has different colors.

When working through the documentation, I found the following page (and others
in appdev):
http://tomcat.apache.org/tomcat-8.0-doc/appdev/installation.html

which seemed a bit outdated, as it mentioned Tomcat 7 and Java 6 (I updated it
to Tomcat 8 and Java 7), but it also mentions CVS (I guess this was written
before SVN existed).

Furthermore, this example web.xml:
http://tomcat.apache.org/tomcat-8.0-doc/appdev/web.xml.txt
declares itself as Web Application 2.3 instead of 3.1.


Note, that I also modified the HTTP and AJP configuration pages so that the
Connector Comparison Chart is now using a table, instead of plain text, as it
is tabular data.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #26 from Mark Thomas ma...@apache.org ---
(In reply to Konstantin Preißer from comment #25)
 Created attachment 30786 [details]
 Part 2 of updated documentation XMLs

Applied. Many thanks.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #23 from Mark Thomas ma...@apache.org ---
(In reply to Konstantin Preißer from comment #17)
 Created attachment 30772 [details]
 First part of updated documentation XMLs

Patch applied with tabs - 4 spaces and trailing whitespace removed.


 I agree that 30% IE  9 users is still too many to switch to XHTML now
 (probably even the 12%).
 (Argh - Old IEs keep being a nightmare for web designers :)

I'm afraid so. Given we'd like to extend this work to docs for all current
versions and the main Tomcat website, 12% is too high a percentage to exclude.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #24 from Konstantin Preißer prei...@web.de ---
Hi Sebb,

(In reply to Sebb from comment #22)
 The display looks a bit odd in IE8 (WinXP).
 
 The images in the top row don't display correctly.
 
 The bottom row is missing the text Comments are disabled for this page at
 the moment. and the horizontal line below it.
 
 Also the Copyright line at the bottom is not centred.
 
 See attached screen prints.

thanks for your feedback about IE8.

When updating the markup and design, I tried to orientate on new Standards
(HTML5 and CSS3) which current browsers support. Unfortunately, IE8 does not
support much of these new standards. Therefore the design looks a bit odd in
IE8.

My thoughts were, If people use a current browser, they will get the correct
layout. However if they use an old browser like IE8, they could get a odd
layout, but the information they search for (documentation) is still
available.

E.g. the JavaScript does not work in IE8 because of the
application/javascript type that is first supported by IE9 (IE8 also lacks a
lot of current JS functionality).

Do you think we should add work-arounds for old browsers like IE8 (e.g.
creating a IE8-CSS that is included with conditional comments)?

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #25 from Konstantin Preißer prei...@web.de ---
Created attachment 30786
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=30786action=edit
Part 2 of updated documentation XMLs

Hi,

here is another part of updated Documentation XMLs.

Note: I also modified the CSS to make tables look a bit better (use different
colors for odd and even rows).

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #18 from Christopher Schultz ch...@christopherschultz.net ---
Re: XHTML (or application/xml), I think it's reasonable to clean-up our
documentation such that the documents would pass an XHTML validator, so that
when the day comes to make the switch, we just switch the content-type and
associated META tags, etc.

My experience is that MSIE 8 and 9 don't completely choke on XML documents, but
it does force the browser into compliance-mode which is definitely nice (e.g.
sane box model, etc.).

Mark: is there anywhere online we can see a sample of the newly-formatted
documentation, or do we have to build it from svn?

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #19 from Mark Thomas ma...@apache.org ---
The CI system makes the latest build of the docs available to view:

http://ci.apache.org/projects/tomcat/tomcat8/docs/

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Re: [Bug 55383] Improve markup and design of Tomcat's HTML pages

[Christopher Schultz]

Mark,

On 8/28/13 10:59 AM, bugzi...@apache.org wrote:
 https://issues.apache.org/bugzilla/show_bug.cgi?id=55383
 
 --- Comment #19 from Mark Thomas ma...@apache.org ---
 The CI system makes the latest build of the docs available to view:
 
 http://ci.apache.org/projects/tomcat/tomcat8/docs/

Thanks for the reference!

-chris



signature.asc
Description: OpenPGP digital signature

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #20 from Sebb s...@apache.org ---
Created attachment 30778
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=30778action=edit
Misaligned display IE8 top

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #21 from Sebb s...@apache.org ---
Created attachment 30779
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=30779action=edit
Misaligned display IE8 bottom

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #22 from Sebb s...@apache.org ---
The display looks a bit odd in IE8 (WinXP).

The images in the top row don't display correctly.

The bottom row is missing the text Comments are disabled for this page at the
moment. and the horizontal line below it.

Also the Copyright line at the bottom is not centred.

See attached screen prints.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #10 from Mark Thomas ma...@apache.org ---
I like it! It is a big improvement on what we have at the moment.

I have your latest patch applied locally after fixing a few minor niggles
(trailing whitespace) with the patch.

The only issue I can see at the moment is that the Tomcat logo doesn't show up.
I'm inclined to commit this patch just as soon as I have fixed the logo issue.

My current view is that once we are happy with this for 8.0.x (I'm pretty much
there - I can't speak for the other committers) we apply similar changes to
7.0.x, 6.0.x and the main Tomcat site (all are built with essentially the same
mechanism).

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #11 from Mark Thomas ma...@apache.org ---
Figured out the logo issue. I missed the new logo you attached.

I renamed it to tomcat.png - no need for the new bit.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #12 from Mark Thomas ma...@apache.org ---
Patches applied to 8.0.x.

I'm leaving this open for further improvements to 8.0.x as well as possible
back-port to 7.0.x, 6.0.x and the website.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #13 from Konstantin Preißer prei...@web.de ---
Hi Mark,
(In reply to Mark Thomas from comment #12)
 Patches applied to 8.0.x.
 
 I'm leaving this open for further improvements to 8.0.x as well as possible
 back-port to 7.0.x, 6.0.x and the website.

Great!

Note, for the documentation there are still some things to do in the various
XML files (like using #One_Two_Three fragment identifiers instead of wrong
#One Two Three (another correct way would be #One%20Two%20Three, but the
XSLT replaces   with _ which also looks better), replacing hard-coded HTML
presentational elements/attributes with CSS, and so on.

I will work on these things, (I just wanted to wait on someone reviewing the
XSLT before I go on).

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #14 from Konstantin Preißer prei...@web.de ---
Hi,

I noticed that when viewing the doc XML source files from SVN over HTTP, like
changlog.xml:
  http://svn.apache.org/repos/asf/tomcat/trunk/webapps/docs/changelog.xml
then the CSS is not applied. Firefox logs an error about wrong MIME type
(text/plain instead of text/css).

This is probably because I forgot to set the correct MIME type on the .css
stylesheet. I think the following change should fix it:

Index: webapps/docs/images/docs-stylesheet.css
===
--- webapps/docs/images/docs-stylesheet.css(revision 1517925)
+++ webapps/docs/images/docs-stylesheet.css(working copy)

Property changes on: webapps/docs/images/docs-stylesheet.css
___
Added: svn:mime-type
## -0,0 +1 ##
+text/css
\ No newline at end of property



BTW, what do you think about switching from HTML to XHTML for the docs of
Tomcat 8 (or next major release)?
I mean:
  - change method attribute on the XSLT from html to xml
  - change file extension of generated output from .html to .xhtml
  - change output MIME type from text/html to application/xhtml+xml

Comparing XHTML over HTML for XSLT-generated content, I currently see the
following points:

Pro:
  - The XSLT processor does not need to know how specific HTML elements
should be outout.
For example, wbr is a void element in HTML5, but currently
the XSLT processor used by Ant/Java outputs wbr/wbr, which is wrong,
as the XSLT processor does not yet implement the current XSLT and XQuery
Serialization 3.0 spec [1].
However, when using XHTML, the browser also uses a XML parser and
therefore XML Syntax can be used (both wbr / or wbr/wbr would be
correct). This avoids the need for the XSLT processor to know how
specific elements should be output (this also concerns elements like
style and script which are output as CDATA in HTML).
  - One does not have to use the about:legacy-compat DOCTYPE (as
work-around until the XSLT processor supports the XLST 3 spec), because
in XHTML there is only Full Standards Mode.
Also, the old-style meta element which includes the encoding in a
Content-Type declaration is not needed.

Contra:
  - IE 9 (released: March 2011; IE 11 will be released soon) is the first IE
to support XHTML, while all other supported browsers like Firefox, Chrome,
Opera etc. support XHTML. IE 8 users will get a download dialog instead
of the page displayed.

Neutral:
  - Because the input (XML files and the XSLT file) is already XML and the
output will be automatically generated, the output syntax
(HTML or XHTML) is not of a concern when editing the source XML files.


However, because the file extension would change from .html to .xhtml, such a
step can probably only be done at a new major release as otherwise existing
links to the pages would stop working.

Using XHTML means that people which use IE 8 or older cannot view the pages.
However, one could think that folks with old browsers stay with old Tomcat
versions, and folks which use the newest Tomcat 8 also use a current IE.  :D


[1]  http://www.w3.org/TR/xslt-xquery-serialization-30/#HTML_DOCTYPE

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #15 from Mark Thomas ma...@apache.org ---
(In reply to Konstantin Preißer from comment #14)
 Hi,
 
 I noticed that when viewing the doc XML source files from SVN over HTTP,
 like changlog.xml:
   http://svn.apache.org/repos/asf/tomcat/trunk/webapps/docs/changelog.xml
 then the CSS is not applied. Firefox logs an error about wrong MIME type
 (text/plain instead of text/css).
 
 This is probably because I forgot to set the correct MIME type on the .css
 stylesheet. I think the following change should fix it:
 
 Index: webapps/docs/images/docs-stylesheet.css
 ===
 --- webapps/docs/images/docs-stylesheet.css   (revision 1517925)
 +++ webapps/docs/images/docs-stylesheet.css   (working copy)
 
 Property changes on: webapps/docs/images/docs-stylesheet.css
 ___
 Added: svn:mime-type
 ## -0,0 +1 ##
 +text/css
 \ No newline at end of property

Fixed.

 BTW, what do you think about switching from HTML to XHTML for the docs of
 Tomcat 8 (or next major release)?

Not yet. Some surveys suggest as many as 30% of browsers are IE  9. That is
too big a chunk of the user base to exclude from the docs.

A better analysis would be to look at the access logs for the Tomcat docs. I'm
unlikely to have the time to do that for a while.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #16 from Rainer Jung rainer.j...@kippdata.de ---
  BTW, what do you think about switching from HTML to XHTML for the docs of
  Tomcat 8 (or next major release)?
 
 Not yet. Some surveys suggest as many as 30% of browsers are IE  9. That is
 too big a chunk of the user base to exclude from the docs.
 
 A better analysis would be to look at the access logs for the Tomcat docs.
 I'm unlikely to have the time to do that for a while.

After about 22 hours of todays logs: MSIE  9 is about 12% of all requests to
the TC vhost access logs and about 10% of the ones to doc content. All
requests means all excluding obvious bots.

Of all MSIE requests, MSIE 5-8 are about 50-60%.

More detailed data:

tomcat.apache.org (US)AllDocs
UACountPctCum PctCountPctCum Pct
Chrome6393241,9%41,9%2489145,3%45,3%
Firefox3673024,1%66,0%1588928,9%74,3%
Safari55183,6%69,6%16703,0%77,3%
AppleWebKit other1650,1%69,7%710,1%77,5%
Opera17311,1%70,9%4910,9%78,4%
Varia112567,4%78,2%25274,6%83,0%
MSIE 54140,3%78,5%80,0%83,0%
MSIE 619731,3%79,8%4650,8%83,8%
MSIE 737602,5%82,3%9681,8%85,6%
MSIE 8125308,2%90,5%38787,1%92,6%
MSIE 975264,9%95,4%23464,3%96,9%
MSIE 1069814,6%100,0%16933,1%100,0%
Sum152516100,0%54897100,0%

MSIE 5-81867712,2%53199,7%

tomcat.apache.org (EU)AllDocs
UACountPctCum PctCountPctCum Pct
Chrome7237243,7%43,7%2692345,5%45,5%
Firefox4020824,3%68,0%1736129,3%74,8%
Safari59203,6%71,5%20363,4%78,2%
AppleWebKit other2000,1%71,7%650,1%78,3%
Opera17451,1%72,7%6201,0%79,4%
Varia107676,5%79,2%25384,3%83,7%
MSIE 56620,4%79,6%50,0%83,7%
MSIE 620051,2%80,8%3140,5%84,2%
MSIE 740482,4%83,3%9771,6%85,9%
MSIE 8127977,7%91,0%41467,0%92,9%
MSIE 980784,9%95,9%24294,1%97,0%
MSIE 1068324,1%100,0%18033,0%100,0%
Sum165634100,0%59217100,0%

MSIE 5-81951211,8%54429,2%

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #17 from Konstantin Preißer prei...@web.de ---
Created attachment 30772
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=30772action=edit
First part of updated documentation XMLs

Hi,

I attached a patch with a first part of updated doc XML files to improve the
HTML markup and follow up to the new XSLT. The changes include:
- Ensure that a p does not contain pre, ul, ol etc. tags.
- Use fragment identifiers that have a space replaced with _.
- Replace presentational tags/attributes with CSS; only use
  table when it is really a table.
- Use CDATA section in source elements so that source code/XML examples
  in the doc XMLs is better readable (I hope this is OK).


Mark and Rainer,
(In reply to Mark Thomas from comment #15)
 Not yet. Some surveys suggest as many as 30% of browsers are IE  9. That is
 too big a chunk of the user base to exclude from the docs.

(In reply to Rainer Jung from comment #16)
 After about 22 hours of todays logs: MSIE  9 is about 12% of all requests
 to the TC vhost access logs and about 10% of the ones to doc content. All
 requests means all excluding obvious bots.

I agree that 30% IE  9 users is still too many to switch to XHTML now
(probably even the 12%).
(Argh - Old IEs keep being a nightmare for web designers :)

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

Konstantin Preißer prei...@web.de changed:

   What|Removed |Added

  Attachment #30741|0   |1
is obsolete||

--- Comment #9 from Konstantin Preißer prei...@web.de ---
Created attachment 30760
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=30760action=edit
Updated patch for improving HTML markup  style in Tomcat docs

Hi,

I updated the patch (again, sorry) to:

• use only 1 data-* attribute for the comments identifier to be retrieved in
JS.
• replace instances of h3a id=../ah3 with h3 id=../h3,
  as it is not needed to use a a element to be able to link to a part of a
  document with a fragment identifier (and a name=... is obsolete).
• remove markup that was only needed for a separate printer-friendly version
  of the docs (which was removed with r601180).
• remove markup that was needed for a status page which was removed with
  r630006.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

Konstantin Preißer prei...@web.de changed:

   What|Removed |Added

  Attachment #30735|0   |1
is obsolete||

--- Comment #6 from Konstantin Preißer prei...@web.de ---
Created attachment 30739
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=30739action=edit
Updated patch for improving HTML markup  style in Tomcat docs

I updated the patch with minor improvements (e.g. replace tabs with spaces,
added AL2 to the new CSS file, further improvements of XSLT contents).

I also changed the JavaScript for the Comments section to annotate
XSLT-generated strings to the script element using the new HTML5 data-*
attributes [1] instead of inserting text directly in JavaScript string
literals, as that would cause problems if the string data contains special
characters.

Furthermore, I added the Tomcat image as normal img with a link, instead of
using it as background-image.

Note that I had to create a new .png image from tomcat.svg as the previous
image was .gif and therefore did not contain alpha transparency.
The patch doesn't contain the image tomcat-new.png as it is binary data - is
it possible to include such files in a SVN patch?


(In reply to Konstantin Preißer from comment #5)
 2) Some issues regarding HTML output:
 
   a) HTML5 specifies for the HTML syntax a recommended doctype of
!DOCTYPE html
 
  For HTML generators that cannot output such a short doctype, it
 provides a Doctype Legacy String [2] as follows:
!DOCTYPE html SYSTEM about:legacy-compat
 
  However, using the doctype-system attribute on the xsl:output
 element, I seem only be able to produce the longer (legacy-compat) doctype,
 but not the short one.

The XSLT and XQuery Serialization 3.0 spec says at point 7.4.6 [2]:

 If the value of the requested HTML version is 5.0, the doctype-public and 
 doctype-system serialization parameters are both absent, the first element 
 node child of the document node that is to be serialized is to be serialized 
 as an HTML element, the local part of the QName of which is equal to the 
 string HTML, without regard to case, and any text node that precedes that 
 element node in document contain only whitespace characters, then the HTML 
 output method MUST output a document type declaration, with no public or 
 system identifier.


That should account for the new HTML5 doctype. However, it seems that the XSLT
serialization library used when building Tomcat with Ant does not yet implement
this, as it doesn't output a Doctype when setting html-version=5.0.


   c) It seems that the library that produces the HTML output does not have a
 current view of the HTML5 void elements (such as br, img etc). For
 example, if I specify wbr / in the XSLT, then the generated HTML output
 will be wbr/wbr, although it should be only wbr, since this is a void
 element (specifying html-version=5.0 on the xsl:output element did not
 change that).

At point 7.1 of the spec [3], it says:

 For HTML5, the void elements are area, base, br, col, command, embed, hr, 
 img, input, keygen, link, meta, param, source, track and wbr.

So just like the above point, the XSLT serializer probably doesn't yet
implement this.


(In reply to Konstantin Preißer from comment #3)
 This has the advantages over using table that
 - the CSS table will not exceed the given width if it has elements that
 are wider (e.g. a pre with long lines of unwrappable text) than the
 table (this should consider the concerns in bug 16579).

To correct me: A table will not exceed the given width when using
table-layout: fixed style. This is also true for table elements.


[1]
http://www.w3.org/TR/html51/dom.html#embedding-custom-non-visible-data-with-the-data-*-attributes
[2] http://www.w3.org/TR/xslt-xquery-serialization-30/#HTML_DOCTYPE
[3] http://www.w3.org/TR/xslt-xquery-serialization-30/#HTML_MARKUP

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #7 from Konstantin Preißer prei...@web.de ---
Created attachment 30740
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=30740action=edit
New Tomcat image to be placed in trunk/webapps/docs/images/ folder

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

Konstantin Preißer prei...@web.de changed:

   What|Removed |Added

  Attachment #30739|0   |1
is obsolete||

--- Comment #8 from Konstantin Preißer prei...@web.de ---
Created attachment 30741
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=30741action=edit
Updated patch for improving HTML markup  style in Tomcat docs

Sorry - the previous patch contained the wrong CSS file.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #5 from Konstantin Preißer prei...@web.de ---
Created attachment 30735
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=30735action=edit
Draft for improved HTML markup of Tomcat Docs

Hi,

as I did not receive negative comments, I tried to apply the new markup and
style to the Tomcat Docs XSLT. :-)

Attached is a patch that uses improved HTML markup and a new CSS stylesheet.

Note that I did not work with XSL Transformation before, so I might have done
strange things in the XSLT file - I would be glad if some XSLT expert could
check that. ;-)


While working in the XSLT file, I found some points on which I'm not clear
(maybe someone can comment on these):

1) The xsl:stylesheet element specifies version=1.0; however, I found that
there's already version 3.0 of the XSLT spec [1] (a working draft), so I
modified the version attribute to 3.0. Is this OK?


2) Some issues regarding HTML output:

  a) HTML5 specifies for the HTML syntax a recommended doctype of
   !DOCTYPE html

 For HTML generators that cannot output such a short doctype, it provides a
Doctype Legacy String [2] as follows:
   !DOCTYPE html SYSTEM about:legacy-compat

 However, using the doctype-system attribute on the xsl:output element,
I seem only be able to produce the longer (legacy-compat) doctype, but not the
short one.

  In XHTML however, a doctype is not needed as there is no quirks mode.


  b) It seems that when using method=html, the generated .html files always
contain the following meta element:
   META http-equiv=Content-Type content=text/html; charset=UTF-8

 so I couldn't use the shorter form:
   meta charset=UTF-8

  c) It seems that the library that produces the HTML output does not have a
current view of the HTML5 void elements (such as br, img etc). For example,
if I specify wbr / in the XSLT, then the generated HTML output will be
wbr/wbr, although it should be only wbr, since this is a void element
(specifying html-version=5.0 on the xsl:output element did not change that).

 Note that for XHTML, the XSLT could simply generate wbr / for void
elements, as well as div / for empty non-void elements, as the browser would
use a XML parser. This would avoid the need for the XSLT processor to know
which elements are void elements.


  I think it could be worth looking at switching to XHTML sometime. However, in
this case IE users would have to use at least IE 9, as that is the first IE to
support XHTML.



With the modified XSLT, the Doc index.html now validates when using the W3X
Non-DTD based validator at http://validator.w3.org/nu/ as opposed to the
previous markup which raised about 63 errors and 10 warnings.


What do you think about it?


[1] http://www.w3.org/TR/xslt-30/
[2] http://www.w3.org/TR/html51/syntax.html#the-doctype

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #3 from Konstantin Preißer prei...@web.de ---
Hi Mark,
(In reply to Mark Thomas from comment #2)
 An overhaul of the docs and site is long overdue as you can tell from the
 age of the duplicate.
 
 Please, go ahead. If you attach proposed patches, I'm happy to review and
 apply them as I am sure other committers will be as well.
 
 One piece of advice I would give is to make small, incremental changes
 rather than a big-bang approach. It is hard to provide guidance to what
 the 'right' size is but the committers will let you know if you are wide of
 the mark.
 
 As you go, please keep in mind the concerns raised in bug 16579.
 
 One comment on the new default homepage, it is (in my view) too cluttered.
 It is a million miles better than what we had before but I think there is
 still scope for improvement.

Ok, thank you.

After reviewing bug 16579, I would like to add to my previous point 3) reg.
table:

You can create a table-like layout purely with CSS (that is, using tags like
div in markup and then use style display: table;, display: table-row
etc).

This has the advantages over using table that
- the HTML markup doesn't misuse a table for layout, which means a user-agent
like a Search Engine can correctly regognize the structure of the document, and
- the CSS table will not exceed the given width if it has elements that are
wider (e.g. a pre with long lines of unwrappable text) than the table (this
should consider the concerns in bug 16579).


There are some other things that came to my attention while I was browsing
through Tomcat's doc:

1) Sometimes there is following markup in HTML (a ul in a p tag): 
p
  Text...
  ul.../ul
  More Text...
/p

For example, r1511000 intruduced such a structure.

Note that this is invalid HTML, as a p element cannot have nested ul and
ol elements. A HTML parser therefore closes the p element as soon as it
sees a ul, and therefore the following /p is incorrect (I think this has
been mentioned already by Konstantin Kolinko).
In XHTML, it is possible to have a DOM like pul.../ul/p, but it is
still not valid XHTML.

The currect markup in this case would be:
pTest.../p
ul.../ul
pMore Text.../p

Please see the HTML5 spec [1] reg. The p element for more info at this issue.


2) The Tomcat Docs use the Comments system (https://comments.apache.org/) by
appeding a JavaScript that will place a comments section there (e.g. at the
bottom at http://tomcat.apache.org/tomcat-8.0-doc/index.html). 

  Note that while the script seems to be intended to be usable in XHTML as it
contains a CDATA-declaration in the script tag, it actually isn't, as it uses
document.write().

  The use of document.write() in JavaScript is strongly discouraged and
actually does not work in XHTML, because it can affect the HTML parser state
while the parser is running [2] which is not possible with XML.

One can use DOM methods like document.createElement(...) to insert elements
when the DOM is ready, in preference to document.write().


That said, I think I can post an idea of a improved markup and tweaked style
for the Tomcat Website and Docs shortly.


[1] http://www.w3.org/TR/html51/grouping-content.html#the-p-element
[2] http://www.w3.org/TR/html51/dom.html#document.write%28%29

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #4 from Konstantin Preißer prei...@web.de ---
Created attachment 30719
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=30719action=edit
Example for new markup and style

I attached an example HTML page with an idea of new HTML markup and tweaked
style, that could be used as template for Tomcat Documentation and Tomcat
Website.

The layout is very similar to the current one (navigation at left-hand-side),
but it should refresh the look so the site doesn't look dated.

There is still room for a completely new layout in the future (like the
proposal from pid), but I do not yet have an idea of how a new layout might
look like (I'm more a programmer than designer), and my favor was to improve
the HTML markup while kepping a similar layout.

The page is HTML but is also viewable as XHTML. It is best viewed at IE10+,
Firefox, Chrome etc. (note that Chrome seems to behave differently than IE10
and Firefox with the font line height).
I'm developing mostly on Windows so I used a Windows font (Lucida Sans Unicode)
as primary font for the site.

Note that older browsers like IE 9 do not show the layout correctly, but
information is still viewable (if people need to view the site with old IE at 
corporate PCs for example).

For the code box, overflow:auto is used, so in the XML example with the long
unwrappable line, browsers should display a scrollbar below the box (but I
needed to use markup like div
class=codeBoxprespan.../span/pre/div for the style).


What do you think?

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

Mark Thomas ma...@apache.org changed:

   What|Removed |Added

 CC||d...@smart.net

--- Comment #1 from Mark Thomas ma...@apache.org ---
*** Bug 16579 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 55383] Improve markup and design of Tomcat's HTML pages

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=55383

--- Comment #2 from Mark Thomas ma...@apache.org ---
An overhaul of the docs and site is long overdue as you can tell from the age
of the duplicate.

Please, go ahead. If you attach proposed patches, I'm happy to review and apply
them as I am sure other committers will be as well.

One piece of advice I would give is to make small, incremental changes rather
than a big-bang approach. It is hard to provide guidance to what the 'right'
size is but the committers will let you know if you are wide of the mark.

As you go, please keep in mind the concerns raised in bug 16579.

One comment on the new default homepage, it is (in my view) too cluttered. It
is a million miles better than what we had before but I think there is still
scope for improvement.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org