Opened 3 years ago

Last modified 7 months ago

#1996 assigned defect

UI prop issues

Reported by: zzz Owned by: sadie
Priority: major Milestone: 0.9.39
Component: apps/console Version: 0.9.30
Keywords: Cc:
Parent Tickets: #2330 Sensitive: no

Description

Big ticket for all my issues after review. Review spanned several versions 0.9.30-5 through -8.

All comments refer to the default theme.

These issues range from objective regressions, to subjective things, to questioning your design intent.
Up to you, no need to respond individually. Disussions may work better on IRC.

Items in first comment.

Subtickets

Attachments (1)

superbasic.tar.2.gz (1.4 KB) - added by str4d 2 years ago.
Super-basic I2PSnark theme for testing CPU usage

Download all attachments as: .zip

Change History (39)

comment:1 Changed 3 years ago by zzz

  • i2psnark: "do not enter" cursor when hovering over standard/open/private radio buttons on tracker section of config page - Why? makes no sense
  • i2psnark: EDIT was monospace before, perhaps it's the larger font that's bugging me? not sure… sorry for the mistake. previous text: monospace font in message box doesn't look good, not an improvement
  • i2psnark: not enough spacing before and after the "mitsubishi" bullets
  • i2psnark: Why is the magnet icon now horizontal, when we have a vertical magnet icon in the console? Go back to the old vertical magnet icon in snark.
  • i2psnark: The big headers at the top (I2PSNARK - FORUM …) are a regression. The letter spacing is too much, the font is too big. And again, icons for everything, are they necessary?
  • i2psnark: The comment table layout is sometimes wonky… haven't nailed it down yet. Placeholder.
  • summary bar: less line spacing in General section than in other sections
  • summary bar: In I2P Internals and Advanced sections, too much inter-line spacing, or not enough spacing above 1st line and below last line, or both
  • summary bar: Still concerned about no text on restart button, users will mistake as page refresh
  • summary bar: The white-over-blue background (or the line between the white and blue) in the <h3> is far too harsh, recommend to going back to the much softer way it was before
  • summary bar: The text-overflow: ellipsis kicks in for hidden services manager in Russian. Why not just go to the next line like before? The tooltip isn't the full text. We don't know how it reads in russian with the ellipsis.
  • summary bar: The dramatic change to the default order may be confusing to users. We should rethink these changes. The most important things are the links to services and other pages… not the bandwidth stats. What was your rationale for the changes?
  • summary bar: The Short General and General sections on /configsidebar should be renamed to something like e.g. "version & uptime"
  • summary bar: You need some way to add the advanced section automatically when configadvanced is set. Otherwise the links have all vanished for the advanced users, and it's not at all obvious how to get them back.
  • summary bar: The icon in the reseed button moves to the left when hovered over. Bug or on purpose?
  • summary bar: The yellow-brown email icon and red magnet icon in the summary bar really stand out from the blue theme everywhere else in the bar. It's not terrible, but doesn't look great either. Another downside of finding existing icons for everything.
  • summary bar: I know you hated the toopies and ripped them all out, but it really helped the eye find the network and tunnel status message boxes. Now the status boxes really blend in with the links, and it's much harder to see at a glance if anything is wrong. I doubt you want to bring back the toopies, but is there anything else that could be done to make those sections pop a little?
  • console: The <h1> is letter spacing is far too big at .12, recommend going back to .08 as it was before. Maybe just a bad merge? I've changed it in the last few months iirc…
  • console: The white background is too bright, or maybe the text is darker, the whole thing is too contrasty, recommend softening it
  • console: Do we really want "hatman" in the background of the config message boxes? I always thought it was dumb, but it wasn't in the light theme before… now it is.
  • console: Recommend replacing the eraser icon, it looks like nothing now, or maybe a pill. Definitely the wrong icon. Maybe an old rectangular chalkboard eraser. Or just a red x or do not enter like we had before.
  • console: /stats is a complete regression, vastly less readable now, the spacing, font choice, and colors are all wrong
  • console: /peers, /profiles, /tunnels, etc. - too little spacing between the flag and the 4-char ident, the 4-char ident font is too big
  • console: /peers, /profiles, /tunnels, etc. - The tables look much worse, it's a combination of the larger font inside the tables and less spacing, it's most obvious on /tunnels
  • console: /peers, /profiles - there's too little contrast between the odd and even table row backgrounds, was better before
  • console: /peers the font in the tables is too big, you've centered all the text in the <td> when it was right-justified before
  • console: /logs the top part is to be copied/pasted, by adding so much inter-line spacing and making different-colored lines in the table, it's less obvious than before. Put a box around it or something, don't make it look like a table.
  • console: /logs you've changed the wording at the top to imply people should copy everything on the page to the bug report, we don't really want that. only the top part.
  • console: /logs putting lines between each log entry in the router logs is a mistake, needless clutter, harder to read
  • console: /logs monospace font in the router logs is a mistake, harder to read
  • console: /config I've apparently lost this battle, but having a different cute little icon in the description for each /config* page is pointless and needless clutter. The cup of coffee with a pencil on /configclients and /configwebapps is particularly bad. Floodfill as a raindrop on /configadvanced is close behind. Deciding every page needs a unique icon, but we aren't icon designers and have only 500 to chose from, is how we got here. We don't need them.
  • console: ditto the proliferation of different icons on buttons. Do we really need, for example, unique icons for the Join Family and Create Family buttons on /configfamily? Seems like needless expansion of the icon set and an attempt to find the perfect and unique icon for each button. The point of an icon in the button is to make it obvious at a glance, esp. for those who don't speak the language in the text. If the icon is meaningless or too complex to figure out at a glance, there's no point. The 10 or so button icons we had before (check, plus, cancel, stop, …) were plenty.
  • console: /config, /configclients, /configservice, /graphs, others? - why [] around the links on the right of the headers? we don't use that to indicate links anywhere else. No value.
  • console: /configplugins - why show the full website and update links for each plugin, rather than the way it was before? needless clutter, no value, a regression.
  • console: Not related to a change from the UI branch, but we may need a separate CSS override for Japanese, or use the Chinese one. The Bold Japanese in the summary bar could be hard to read. Also, though not as bad, Korean too.
  • console: In Chinese, the cancel and apply buttons are different heights on some pages, e.g. /configui
  • console: On the buttons with circular arrows (restart, reload, check for updates…) they have a cute animation when you hover over them, but none of the other icons do, so it isn't consistent. Is the animation necessary or helpful? Seems pointless flash.
  • console: You added help-faq.jsi, but there's no mechanism to translate it, like the other help*.jsi files, although that was promised when you split the help into .jsi files 3 ½ years ago. Now we have more untranslated content… we shouldn't be adding more.
  • i2ptunnel config: The way you've reworked it, I'm not sure the popups over the checkboxes and radio buttons are necessary any more. Rather than add more information, they are shorter than the text next to the box, and unhelpful.
  • i2ptunnel config: signature type DSA-SHA1 has a 'default' tooltip which probably isn't right
  • i2ptunnel config and registration helper: The <h2> section headings aren't big enough or don't have enough spacing. There's no visual separation between the two sections. The registration helper page in particular is a big regression.
  • i2ptunnel config and registration helper: Having huge textareas (full width) for the authentication strings is unnecessary and adds clutter.
  • susidns: A link to the image generator on the details page seems pointless. The use case seems extremely rare.
  • susidns: The destination is no longer in a scrollable text box on the details page, why? No need to clutter up the page with the whole string.
  • susidns: Is a lifesaving ring really the best icon for an addresshelper link? Do we need an icon for absolutely everything? looks silly.
  • susidns: Putting each filter letter in a box takes up way too much horizontal space, wraps to second line, looks bad.
  • susidns: Putting an identicon on each line on the index page looks cool, but ultimately pointless, and might take far too long to generate on an RPi. I haven't tested it yet. Needs testing, and possibly disable on RPi.
  • susidns: Recommend replacing the eraser icon, it looks like nothing now (see console)
  • imagegen: That whole cleanup, while pretty, seems pointless. See the susidns comment. The use case seems extremely rare. The page was just there previously for debugging, and as a landing page from /configwebapp. There's no mechanism for translations.
Last edited 3 years ago by zzz (previous) (diff)

comment:2 Changed 3 years ago by zzz

Most of the above testing using Firefox 53.0.2 on a 24" LCD 1920x1080, browser width about 1200px (2/3 screen width)

Fix the ones you agree with, ignore the ones you don't. Not worth the time to debate.

Only a few more for now. Will keep testing:

  • i2psnark: in show-peers mode, the peers aren't right-indented like they were before. looks worse. Before, with them over to the right, each torrent was visually separated. That was very much on purpose.
  • i2psnark: in show-peers mode, the per-peer terminal icons and "peer attached to swarm" tooltips are useless clutter.
  • i2psnark: on the totals line, why is the 4-char Dest in yellow? Clashes with everything else, we don't use that color anywhere else on the page. Was white before.
  • console: On the home page, do we really want to flip the order of the "configuration" and "hidden services" sections? I believe we flipped it the other way years ago, on the principle that the main point of the console is a gateway to popular hidden services; that the rest of the console is rarely-used configuration stuff, and that the hidden service links should be emphasized. Have we changed our mind (again) on that?
  • console: On the home page, the bandwidth section is also now part of the default and near the top (as in the full summary bar, see above comment) Why have you elevated the importance of the bandwidth box so much? Seems like a tech doodad, not useful for most users.

comment:3 Changed 3 years ago by zzz

  • summary bar: After clicking shutdown, the 'shutdown immediately' and 'cancel shutdown' buttons aren't where they used to be, they are now way down at the bottom. Seems like a usability regression, they should be where the shutdown button was when it was clicked, upper middle.

comment:4 Changed 3 years ago by zzz

re: comment 3, now the regular shutdown and restart buttons moved to the bottom too. not sure if that was a recent change, or the result of me resetting the default so I could see what it looked like, or what. I'd like to hear the principles that drove your reordering of the summary bar so we can discuss whether the changes have accomp[lished them, or whether they are worth the possible confusion for existing users.

comment:5 Changed 3 years ago by zzz

(same browser as in comment 2)

The change from   to   in DataHelper?.formatSize2() causes the logfilesize param on /configlogging to be rejected by the XSS filter, so it can't be changed. Now, by the time it's logged in the router log, it's looks like a regular space… but a hex dump of the logs shows E2 80 AF so it's in there… looking at view-source, it's in the form with the  , not as a space, so it's presumably coming back in the POST like that. There may or may not be other places where this happens, I don't know.

In addition, there are some places in the code base where we convert the   back to a regular space, and a quick search through the code base for 8239 doesn't find any places where you've fixed those.

In the past we've had trouble with   support in browsers… esp. Opera and text-mode browsers? Don't know if &8239; would be better or worse, or if things have improved in recent years. Briefly tested lynx where it displayed correctly, and dillo where it did not.

I see the comment in the code that you made the change because it looks better… if we keep it, will have to really go through everything to find and fix what it broke.

(edit - corrected info on unicode in logs)

Last edited 3 years ago by zzz (previous) (diff)

comment:6 Changed 3 years ago by lazygravy

Some notes - I like the overall redesign :)

Mac 10.12.5 issues (Chrome/FF):

  • general
    • not really in scope of your ticket - but maybe less should be shown while not in advanced mode?
    • Janky feeling scrolling (only in Chrome? ~fps while geti2p.net is 60fps)
      • quick debugging shows that the entire page is repainted
      • the firefox repainting tools don't know a full repaint
    • The reset and shutdown buttons overlap (ask me for a screenshot)
    • Some of the icons are blurry (this screen is outrageously high quality, so it might not matter to 99% of users)
    • forum.i2p is down… time to get rid of it? :(
  • /configwebapps
    • when screen is not wide enough, tabs overlap (not sure if intentional)
  • /graphs
    • upper right graph is not the same size as the rest
  • /home
    • "Customize Home Page" → "Edit Home Page" to get rid of ellipses?
  • /help
    • A lot of the router to router links point to the .jsp version (not a bug but worth mentioning?)
  • /console
    • The star bullet looks a little misaligned
  • Tunnel Wizard
    • the inner div comes out of the outer one

comment:7 Changed 3 years ago by zzz

re: forum.i2p, I have the changes pending in my workspace to remove the links, I will check them in later in the cycle if the site doesn't reappear.

comment:8 Changed 3 years ago by zzz

re: tabs overlapping on /config*, it is intentional, and they overlapped before the UI prop. However, it looked more intentional before, as the tabs actually looked like tabs, and the effect was more subtle. Now, it looks more accidental. Squaring everything off was obviously a choice. Not sure if the new look is completely successful.. would be nice to hear more comments.

re: /graphs the combined bandwidth graph has always been a different size. It's a compromise. We don't know in advance the size of the legends, so if we scale the whole image it looks crappy.

re: ellipses on 'customize home page' - I can't make that happen on firefox (see above for version info), but if there's ellipses in the CSS for the home page icons, I don't think that's a good idea. See also 11th item in comment 1 above re: ellipsis CSS in summary bar.

comment:9 Changed 3 years ago by zzz

Priority: minorblocker
  • You changed the default graph width from 250 to 400. I think that messes up the balance of the graphs page when it's two-column, and on a basic laptop it's one column which looks worse. It seems like str4d has some super-expensive hi-def display or laptop that nobody else has, and he's optimizing it for his setup? Is that why there's all the change in icons, but they don't look any different to me? I'm beginning to understand why it seems like there's so many regressions to me.. perhaps it's just different displays. To be researched more.

Also, changing priority to blocker, not because most of these aren't minor, but because they need to be triaged and at least something in here probably needs to be fixed before the release.

comment:10 Changed 3 years ago by zzz

  • on /logs, if there are unavailable crypto warnings (from LogsHelper?.getUnvailableCrypto()) they are not formatted inside a <tr><td>, and are displayed floated up above the table without the correct styling. (Tested with java 9)

partial from view-source:

<table id="enviro"><tbody>
<tr><td><b>I2P version:</b></td><td>0.9.30-10</td></tr>
<tr><td><b>Java version:</b></td><td>Oracle Corporation 9-Ubuntu (OpenJDK Runtime Environment 9-Ubuntu+0-9b161-1)</td></tr>
 
 
<b>Crypto:</b> ECDSA_SHA256_P256 unavailable<br><b>Crypto:</b> ECDSA_SHA384_P384 unavailable<br><b>Crypto:</b> ECDSA_SHA512_P521 unavailable<br>
<tr><td><b>Wrapper version:</b></td><td>3.5.30</td></tr>

comment:11 Changed 3 years ago by zzz

re: comment 5, I see you changed formatSize2() but not formatDuration2(). Why not keep them consistent? My comment about places in the code that change the nbsp back to a regular space may apply only to formatDuration2(), not sure, haven't searched. The duration call may be much more common. At this point it may make the most sense to revert the size change. If you change the duration also, will need more testing and searching for similar issues.

comment:12 Changed 3 years ago by zzz

Did some brief testing on a Macbook Safari 10.1.1 and a Windows netbook IE 11.0.9600.18666
Did not see anything that was very different from my firefox testing above.
On safari, the thin space (comments 5 and 11 above) is really really thin.
Will continue posting test results as I continue.

comment:13 Changed 3 years ago by zzz

i2psnark: the completion graph bars show the completion % or bytes when you hover over it, but it takes a long time, almost a second. I don't know if that's an intentional fade-in effect, or the browser is taking a long time to do it. Unfortunately, the result is the display is really laggy when you move up and down over a bunch of peers (in show-peers mode) or the files (on the details page). Need to do something to speed it up, or else get rid of the graphs and go back to the text only. The graphs aren't bad but combined with the fade-in fade-out it's too much.

comment:14 Changed 3 years ago by zzz

i2ptunnel edit page: When I click in an editable box in the form, the text turns bold. Is that on purpose? It's bizarre. Since it changes the size of the text, the cursor jumps over. Particularly strange in a box with lots of text (try putting a bunch of b32s in the access list). Almost tolerable in the boxes with only a couple of digits (e.g. inbound connection limits). But ultimately pointless, and harmful in big textareas.

comment:15 Changed 3 years ago by zzz

re: i2psnark SimpleDateFormat?

-                SimpleDateFormat fmt = new SimpleDateFormat("yyyy-MM-dd HH:mm");
+                // needs locale configured for automatic translation
+                SimpleDateFormat fmt = new SimpleDateFormat("HH:mm, EEEE dd MMMM yyyy");

I've tried, not always successfully, to keep date displays consistent in the console. We've never used your new format before.

The issue is not only translation but also formatting. We don't ever set the Locale in the JVM. You can either create a locale for the current translation setting (which we don't usually do) or use the JVM's Locale. To test with other locales, start the JVM with a command line option.

For simple format specs, it's been fine to specify them, but for longer ones, we risk it looking strange to some users as we impose what looks "right" to us. In that case, it's better to use the Locale-specific builtins. The one I've settled on that works well, and we use in several places, is

SimpleDateFormat fmt = (SimpleDateFormat) DateFormat.getDateTimeInstance(DateFormat.SHORT, DateFormat.MEDIUM);

You can write a quick program to see what SHORT, MEDIUM, and LONG look like in your locale (or all supported locales) for time and date. I may even be able to find the one I did quite a while ago.

It's a compromise, but the more you try to tweak complex formats to make everybody happy the more you're going to fail. This long one with a comma and name-of-day (EEEE) is a fail.

If you like we can make a pass through the whole codebase grepping for SimpleDateFormat? and trying to ensure even more consistency in formatting, translation, and timezone, but for now let's not add yet another one.

comment:16 Changed 3 years ago by slumlord

Things look generally good, my comments:

summary bar: In I2P Internals and Advanced sections, too much inter-line
spacing, or not enough spacing above 1st line and below last line, or both

Spacing & layout can be improved (imo)
For example, the 'I2P INTERNALS' link. The 'Hidden Services Manager' link could
be placed on the first line, with 'Addressbook' & 'Graphs' on the second line.
I haven't tried this, it may look better?

summary bar: Still concerned about no text on restart button, users will
mistake as page refresh

Text on the restart/shutdown buttons would be more explanatory
Not consistent across themes, 'dark' theme has text without icons.

summary bar: The dramatic change to the default order may be confusing to
users. We should rethink these changes. The most important things are the links
to services and other pages… not the bandwidth stats. What was your rationale
for the changes?

I noticed this too. I think the reason is that bandwidth stats are shown on
/home page now.

Animation on buttons

A hover state would be useful, I'm not so sure about an animation.

/logs page

Hover state for 'I2P Version and Running Environment' is poorly done. Text area
shrinks upon hover which does not look nice.
I think monospace font for logs is good.

i2psnark

In the 'dark' theme, 4-char dest is in a shade of green that does not match
with the rest of the theme (to my eyes, anyways). Padding is insufficient, the
'j' character blends with the background.

comment:17 Changed 3 years ago by Obscuratus

i2psnark comments/rating:

After the first comment is entered, the alignment for the portion of the UI for entering comments all gets squashed over to the right-hand side.

comment:18 Changed 3 years ago by Obscuratus

I've been running 0.9.30-9 and 0.9.30-10 on my testing network, and I haven't run across any major functionality regressions.

One dumb thing I did on my part that I'm sure several other users will encounter:

Make sure and flush your browser cache before testing. And, when people report odd behavior in the UI after we release 0.9.31, make sure they've flushed their browser's cache.

I've been trying to review the issues brought up in previous comments, and the following issues jumped out for me.

i2psnark: "do not enter" cursor when hovering over standard/open/private radio buttons on tracker
section of config page - Why? makes no sense

I agree. It didn't make sense.

i2psnark: Why is the magnet icon now horizontal, when we have a vertical magnet icon
in the console? Go back to the old vertical magnet icon in snark.

I agree.

i2psnark: The big headers at the top (I2PSNARK - FORUM …) are a regression. The letter
spacing is too much, the font is too big. And again, icons for everything, are they necessary?

I agree, this looks really large on my browser.

summary bar: Still concerned about no text on restart button, users will mistake as page refresh

I agree. This one is probably important. People are going to mistake this button for some kind of refresh.

console: /stats is a complete regression, vastly less readable now, the spacing, font choice,
and colors are all wrong

I disagree (I'm reviewing 0.9.30-10, so I'm not sure if there have been changes). I found the look for the stats page to be cleaner, and the font was more readable for me.

comment:19 Changed 3 years ago by zzz

re: flushing cache (or simply shift-reload or ctl-reload depending on browser), yes that is important for dev build testers.

As the CSS links are versioned with the base router version, this will not be necessary for the release.

comment:20 Changed 3 years ago by Eche|on

Ok, added ticket 2001 and 2002 with issues:

  • i2psnark page in firefox 45.9.0-ESR on debian stretch it blocks the firefox random, no special action seen, happend after add torrent, stopped torrent or removed a torrent so far, non on first try, but after some time. Maybe associated with the progress bars
  • i2psnark page in konqueror 4.14.25 (debian stretch) does work far better than in Firefox (no lock ups), but the add torrent or create torrent buttons do not work over here

comment:21 Changed 3 years ago by zzz

Moving my mouse up and down over the snark torrents table drives firefox to 80-100% CPU. Whatever fancy stuff you've added must be scaled back drastically. See also comments 13 and 20 above and #2001.

comment:22 Changed 3 years ago by zzz

re: 8239 (comments 5 and 11), I still recommend reverting that change; should you choose to keep it, the fix for the XSS would be in ConfigLoggingHelper?

comment:23 Changed 3 years ago by slumlord

Comment 21:

On Qubes OS running a fedora-23 VM, I can reproduce zzz's observation in comment 21 with Chromium 54.0.2840.90 Fedora Project

Randomly moving my mouse around the i2psnark torrent list causes CPU usage to spike. I have 2 chromium-broswer processes, one shows 160% CPU usage in top, the other shows 60%. I think a system under low load will handle this reasonably well but a system under heavier load (or a system with fewer resources) will not.

edit: updated to -11 and issue still exists.

Last edited 3 years ago by slumlord (previous) (diff)

comment:24 Changed 2 years ago by slumlord

Updated to -13, high CPU usage on mouse-over across screen is still present. It seems like there are 2 mouseover states? One for mouse-over on the table's row and another for mouse-over on the text (as well as another for mouse-over on the button to the right). If I move my mouse across the screen, around the location of the torrent names, a mouse-over state is rendered twice for each torrent. I'm on the dark theme.

An update on a previous comment: background on the destination ID is insufficient only if text-size is reduced to around 75%, it looks okay at regular size.

comment:25 Changed 2 years ago by slumlord

Tried testing on the default theme as suggested by str4d, CPU behavior is the same. I see high CPU usage on mouseovers — CPU usage is high even when only a few torrents are loaded.

I will try testing my browser on a simple webpage with similar CSS to see if I can reproduce this.

comment:26 Changed 2 years ago by slumlord

I was working on something else but this may be a simple test: http://crypthost.i2p/Wiki/bin/view/SCENEi2P/DemocracyNow#Archives

that page seems okay, I think

Changed 2 years ago by str4d

Attachment: superbasic.tar.2.gz added

Super-basic I2PSnark theme for testing CPU usage

comment:27 Changed 2 years ago by str4d

Replying to slumlord:

Tried testing on the default theme as suggested by str4d, CPU behavior is the same. I see high CPU usage on mouseovers — CPU usage is high even when only a few torrents are loaded.

It seems that the CPU spike is caused by the browser more than anything related to the CSS. I tried the old ubergine theme, and was able to obtain 100% spikes with repetitive mouse movement over the torrents list. I was also able to reproduce it with a super-basic theme (see above attachment - Trac failed on first upload, seems to have caused the odd file extension).

Note that in both cases the UI remains responsive, which was not the case before the lag issue with the ubergine theme was addressed. So my conclusion is that, at least in Firefox, rapidly moving over a bunch of links (in a table) will cause CPU spike regardless of how heavy the CSS styling is. Seems like a browser issue more than anything else.

comment:28 Changed 2 years ago by slumlord

Thank you for the update, str4d.

I commented on this earlier:

The order of the links under the I2P Internals section has changed, it looks like it was arranged alphabetically. Under the current layout, it is not clear that 'Hidden Services Manager' is 1 link and not 3 until a mouse-over. This link was previously placed at the bottom row, which I personally preferred. I'm not sure what the reason was for re-arranging the links. The spacing used in the link vs. the spacing between links seems almost the same? This does not seem like the best layout as it is a little visually confusing.

Last edited 2 years ago by slumlord (previous) (diff)

comment:29 Changed 2 years ago by slumlord

I am unable to find the button for 'Show Local Identity' now,

http://localhost:7657/netdb?r=. this page

It used to be in the 'General' section

comment:30 Changed 2 years ago by zzz

Followup re: date formatting #2016
Followup re: translated strings #2017

comment:31 Changed 2 years ago by zzz

more after review:

i2psnark: Why add tracker URLs to each tracker under 'create torrent'? Useless clutter, not helpful to user, info is available on config page, we added names for each tracker for a reason - so we wouldn't have to display the URL.

i2psnark: You restored the long-commented-out meta refresh, for when there's no js. Having auto-refresh of a whole page that contains a form is really fugly when it refreshes while a user is entering something. Why bring this back? Was anybody asking for it? Seems like a bad idea.

susidns: why remove resetting of the filter/begin/end params in the private/master/router links at the top of addressbook.jsp (and other pages too? not sure). They were there for a reason, when you change books we want to clear the params, to not do it is very confusing to the user.

summary bar: still concerned about seemingly arbitrary changes, including drastic reordering, which aren't improvements to my eye

summary bar: still no labels on restart and shutdown buttons. I though you were going to restore them? Massive potential for confusion here.

more comments to follow, perhaps.

comment:32 Changed 2 years ago by zzz

Milestone: 0.9.310.9.32
Priority: blockermajor
Status: newopen

Not sure what you want to do with this ticket now that .31 is out. Moving from 31 to 32 and from blocker to major and from new to open.

comment:33 Changed 22 months ago by zzz

Milestone: 0.9.320.9.34

Please finish whatever you're going to do for .34 and close this ticket. Let's not make this a multi-year effort.

comment:34 Changed 19 months ago by zzz

Milestone: 0.9.340.9.36
Owner: changed from str4d to zzz
Status: openaccepted

comment:35 Changed 14 months ago by zzz

Milestone: 0.9.360.9.38
Parent Tickets: 2330

comment:36 Changed 11 months ago by zzz

Milestone: 0.9.380.9.39

snark responsiveness issue (comments 23-27) moved to separate ticket #2375

comment:37 Changed 8 months ago by zzz

Owner: changed from zzz to sadie
Status: acceptedassigned

comment:38 Changed 7 months ago by sadie

summary bar: The white-over-blue background (or the line between the white and blue) in the <h3> is far too harsh, recommend to going back to the much softer way it was before

  • i2poffwhite: #f6f6fa suggested, which the colour used on the I2P site

summary bar: The icon in the reseed button moves to the left when hovered over. Bug or on purpose?

  • has this been fixed? I don't see it happening for me

summary bar: The dramatic change to the default order may be confusing to users. We should rethink these changes. The most important things are the links to services and other pages… not the bandwidth stats. What was your rationale for the changes?

  • agreed, i2p services, internals and the help section are more important.

Ordering could be Router info, I2P services, I2P Internals, Help & FAQ followed by bandwidth, network status, reseed, peers, tunnels, tunnels status, restart/stop, local tunnels/shared clients.

summary bar: You need some way to add the advanced section automatically when configadvanced is set. Otherwise the links have all vanished for the advanced users, and it's not at all obvious how to get them back.

agreed

susidns needs less icons

Note: See TracTickets for help on using tickets.