> On July 25, 2013, 11:55 p.m., Ben Mahler wrote: > > src/webui/master/static/browse.html, lines 79-82 > > <https://reviews.apache.org/r/12887/diff/1/?file=326197#file326197line79> > > > > Does this still render well when there are no files?
I added a "No files in this directory." message when the file list is empty. - Ross ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12887/#review23894 ----------------------------------------------------------- On July 26, 2013, 7:03 p.m., Ross Allen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12887/ > ----------------------------------------------------------- > > (Updated July 26, 2013, 7:03 p.m.) > > > Review request for mesos. > > > Repository: mesos-git > > > Description > ------- > > Style /browse to distinguish folders > > * Add small folder icons to the left of folders to distinguish them > from other files. > * Right align Size, Nlink, and mtime columns. > * Remove 'name' title since it is apparent > * Order files first by file type, i.e. folder or not folder, and then > by name. > > Review: http://reviews.apache.org/r/12887 > > > Diffs > ----- > > src/webui/master/static/browse.html > 366e99978554916feb7be962d4f1df8fd93773cc > > Diff: https://reviews.apache.org/r/12887/diff/ > > > Testing > ------- > > > File Attachments > ---------------- > > Pre-commit browse > > https://reviews.apache.org/media/uploaded/files/2013/07/24/Screen_Shot_2013-07-23_at_5.13.32_PM.png > Post-commit browse > > https://reviews.apache.org/media/uploaded/files/2013/07/24/Screen_Shot_2013-07-23_at_5.09.23_PM.png > Right-aligned headers > > https://reviews.apache.org/media/uploaded/files/2013/07/26/Screen_Shot_2013-07-26_at_11.56.18_AM.png > Headers aligned same as column > > https://reviews.apache.org/media/uploaded/files/2013/07/26/Screen_Shot_2013-07-26_at_11.56.25_AM.png > > > Thanks, > > Ross Allen > >
