Hi On Mon, Jul 24, 2017 at 11:34 AM, Harshal Dhumal < harshal.dhu...@enterprisedb.com> wrote:
> Hi, > > Please find attached patch to set default layout of file listing as a list > in file manager. > Also replaced alertify with out custom alertifywrapper in file manager > utils.js > This isn't a bad idea on the face of it, but there are some things to fix/consider: - The HTML file seems to be missing translation markers. Can you add them throughout please? - We should save the users preference in the config database. - The grid view seems to underline the file size and for no apparent reason change the mouse cursor to ? on mouseover. Let's change the text style to be consistent and get rid of the mouseover. Thanks! -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company
