Modern theme for about:networking

ASSIGNED
Assigned to

Status

SeaMonkey
Themes
ASSIGNED
2 years ago
2 years ago

People

(Reporter: Philip Chee, Assigned: Philip Chee)

Tracking

({modern})

SeaMonkey 2.47 Branch
modern

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

5.19 KB, patch
Philip Chee
: review?
neil@parkwaycc.co.uk
Ian Neal
: review+
rsx11m
: feedback+
Details | Diff | Splinter Review
3.87 KB, patch
Philip Chee
: feedback?
neil@parkwaycc.co.uk
Philip Chee
: feedback?
Ian Neal
rsx11m
: feedback+
Details | Diff | Splinter Review
(Assignee)

Description

2 years ago
about:networking is un-themed in modern.
(Assignee)

Comment 1

2 years ago
Created attachment 8785444 [details] [diff] [review]
Patch v1 css for about:networking (modern)

There is this nifty toolkit about page which doesn't yet have any modern theming so here is one.
Attachment #8785444 - Flags: ui-review?(rsx11m.pub)
Attachment #8785444 - Flags: review?(neil)
Attachment #8785444 - Flags: review?(iann_bugzilla)
(Assignee)

Comment 2

2 years ago
Created attachment 8785446 [details] [diff] [review]
Diff with toolkit windows aboutNetworking.css
Attachment #8785446 - Flags: feedback?(rsx11m.pub)
Attachment #8785446 - Flags: feedback?(neil)
Attachment #8785446 - Flags: feedback?(iann_bugzilla)
Looks good at first glance. Could you compare the font used to the one in about:buildconfig. The regular font for the list items might look a little more fuzzy here but not sure.

Comment 4

2 years ago
Comment on attachment 8785444 [details] [diff] [review]
Patch v1 css for about:networking (modern)

Seems a good starting point, for some reason there is always a scroll bar.
Should there be a differentiation of DNS Lookup from the other buttons (like there is in classic)?
Does classic also need some formatting? e.g. the icon on the warning dialog is partly off screen
Attachment #8785444 - Flags: review?(iann_bugzilla) → review+
(Assignee)

Comment 5

2 years ago
(In reply to Frank-Rainer Grahl from comment #3)
> Looks good at first glance. Could you compare the font used to the one in
> about:buildconfig. The regular font for the list items might look a little
> more fuzzy here but not sure.

both have font: message-box; which maps to font-family: -moz-use-system-font; which on my platform maps to Tahoma 11px on both pages. Might be different if you are on Linux
>>
both have font: message-box; which maps to font-family: -moz-use-system-font; which on my platform maps to Tahoma 11px on both pages. Might be different if you are on Linux
<<

No then its good. I just thought it might look a little bit more fuzzy but when same font it seems its just the different words which causes this. May need to adjust Clear Type.

Comment 7

2 years ago
Comment on attachment 8785444 [details] [diff] [review]
Patch v1 css for about:networking (modern)

Looks good in general, and I don't see any font issues on Linux with GTK3 (it's the same font as used with other about: pages, like the Troubleshooting page).

A couple of remarks on the top row of buttons:

 - There is no or little padding of the text to the left; should the text be centered?
 - In contrast to other buttons, there is no visual "pushed" feedback, they stay the same.
 - Similarly, it's difficult to tell which button is active right now, they change style in classic (ok, the title of the tab makes it unambiguous).

(In reply to Ian Neal from comment #4)
> Should there be a differentiation of DNS Lookup from the other buttons (like
> there is in classic)?

I'd say yes. The DNS Lookup is an interactive tool without regard of the current status of SeaMonkey (other than DNS availability) whereas the other functions are strictly queries on currently open connections.

> Does classic also need some formatting? e.g. the icon on the warning dialog
> is partly off screen

I've noticed the same in Firefox 48.0, also there is no padding of the text on a small screen, which is irritating. This should be a bug against toolkit.
Attachment #8785444 - Flags: ui-review?(rsx11m.pub) → feedback+

Comment 8

2 years ago
Comment on attachment 8785446 [details] [diff] [review]
Diff with toolkit windows aboutNetworking.css

>+#categories > .category
>+{
>+  -moz-appearance: button;
>+  background-color: #AAB6C4;
>+  background-clip: padding-box;
>+  color: #000000;

Looks like the "pushing" styles may need to be added here and overwrite the default background-color and color?

> #categories hr {
>-  border-top-color: rgba(255,255,255,0.15);
>+  display: none;
> }

This should be the separator for the DNS Lookup button, but it would have to be vertically oriented in modern.

>-.title::before {
>-  content: "";
>-  left: -2.3em;
>-  top: 0;
>-  position: absolute;
>-  display: block;
>-  width: 1.6em;
>-  height: 1.6em;
>-  background: url("chrome://global/skin/icons/warning.svg") no-repeat left center;
>-  background-size: 1.6em;
>-}

The -2.3em with absolute positioning relative to the title are probably the reason that the icon gets pushed outside of the window in classic.
Attachment #8785446 - Flags: feedback?(rsx11m.pub) → feedback+
You need to log in before you can comment on or make changes to this bug.