On 13/07/16 06:50, Niklas Laxström wrote:
Many good and bad changes I can live with. One thing I will miss:
* Columns setting no longer wraps the diff lines. Horizontal scroll is
now unavoidable and cumbersome due to having to use either the mouse
or the arrow keys to move the cursor on the line.
Yeah, if it's possible to make these wrap instead of horizontal
scrolling, that would be great. There is way too much scrolling
everywhere, even on large displays.
Here's a bunch of other design things I'm hoping we can fix, since
they're all a bit glaring:
* Diffs - using icons for back to change, previous file, next file is
very unclear and hard to find; new users will not see them and have
any idea what to do with them. The current textual 'up to change',
previous/next file is immediately clear and would be great to
maintain. (The side-by-side and unified diff icons are also very
strange, like fuzzy rgb blobs, but at least they're a bit clearer.
Sort of.)
* Commit message - limiting the height doesn't really help anything
since if there's a long commit message, presumably it's long for a
reason and we still want to see it. And it really makes it worse on
small screens where we need to scroll through everything anyway, so
the fewer separate things we need to scroll through, the better.
* Diff preferences - white text on black background is very bad,
especially when it suddenly appears on top of an interface that uses
black text on white.
* Padding - please more. Things should not be shoved up against other
things. It makes them hard to pick out and read things at a glance,
and the entire thing looks a lot messier and overly busy than it
actually is; there should be padding around each bit of content (esp
the commit message stuff), and at the end of blocks in order to
distinguish them from the blocks that come after.
* The very narrow scrollbars are weird and unnecessary, and hard to
use. Considering most people are probably either on a system that
already has similar by default (and hides them when not in use), or
have large system scrollbars for a reason (hi), would it be possible
to just remove that?
* hrs probably need styles
* Solid black lines are generally bad
* Is there any way we can put the different file opening options back
on the main change page? The view side-by-side diff, view unified
diff, and now the edit file option too, instead of only having the
filename as a link? It opening up to whatever mode was last open is
really weird behaviour, especially with edit, since closing the edit
mode always dumps the user back onto the main change page instead of
going back to viewing the diff, and then when they click on the file
again it takes them back into edit mode, which they don't want, and
the obvious path at that point is 'close', even though the only way
back to the diffs for anything at that point is the rather ambiguous
icons in the corner of the edit thing.
Assuming it's at least pretty trivial to append css, here's a quick fix
for the commit message box:
.com-google-gerrit-client-change-CommitBox_BinderImpl_GenCss_style-collapsed
.com-google-gerrit-client-change-CommitBox_BinderImpl_GenCss_style-scroll {
padding: 1em;
height: auto;
}
.com-google-gerrit-client-change-CommitBox_BinderImpl_GenCss_style-header {
padding: 1em;
}
Quick fix for the diff preferences:
.com-google-gerrit-client-diff-PreferencesBox_BinderImpl_GenCss_style-dialog
{
background: rgba( 240, 240, 240, .9 );
color: #000;
border: solid 1px #ccc;
text-shadow: none;
}
.com-google-gerrit-client-diff-PreferencesBox_BinderImpl_GenCss_style-table
td,
.com-google-gerrit-client-diff-PreferencesBox_BinderImpl_GenCss_style-table
th {
color: #000;
}
Maybe?
hr {
border: solid 1px #ccc;
}
Note that all colours are made up on the spot, I don't know anything
about how gerrit actually comes up with these crazy class names, so
nothing is really guaranteed with these snippets, etc etc. But please,
if we can make this better, that would be great. Some of the new
features are already great. Also the other things.
-I
_______________________________________________
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l