Re: [pgAdmin][RM5990]: Update the style guide on pgAdmin4 website

2020-09-10 Thread Nikhil Mohite
Hi Team, I have updated UI for the style guide, details as follow: 1. Resolved all UI changes suggested by Aradhana. 2. Added labels for query-tool and tree-views icons in iconography. PFA patch v7 for the same. If required any changes please let me know. Regards, Nikhil Mohite. On Wed, Aug

Re: [pgAdmin][RM5990]: Update the style guide on pgAdmin4 website

2020-08-26 Thread Aradhana Birewar
Hi Nikhil, I was going through the style guide, I have a few observations that I would like to share with you. 1. In the Typography section, for font size text, the Roboto font is not getting applied. [image: image.png] 2. In the Dropdown section, the expanded dropdown image appe

Re: [pgAdmin][RM5990]: Update the style guide on pgAdmin4 website

2020-08-26 Thread Dave Page
Thanks, applied with a few changes for consistency in the titles and 404 handling. On Wed, Aug 26, 2020 at 8:23 AM Nikhil Mohite < nikhil.moh...@enterprisedb.com> wrote: > Hi Dave/Team, > > Updated the implementation for the set title of the page, for this, I have > done a few changes in the flow

Re: [pgAdmin][RM5990]: Update the style guide on pgAdmin4 website

2020-08-26 Thread Nikhil Mohite
Hi Dave/Team, Updated the implementation for the set title of the page, for this, I have done a few changes in the flow. 1. Earlier we are using {% include %} for showing pages in the base page (styleguide/index.html) for a style guide but in the Django template we can't use {% block %} statement

Re: [pgAdmin][RM5990]: Update the style guide on pgAdmin4 website

2020-08-25 Thread Dave Page
Hi On Tue, Aug 25, 2020 at 12:49 PM Nikhil Mohite < nikhil.moh...@enterprisedb.com> wrote: > > PFA v5 patch for the same. > > if required any updates in this please let me know. > Hmm, yeah - that doesn't work for me I'm afraid; it's mixing content with code. The titles should be defined in the

Re: [pgAdmin][RM5990]: Update the style guide on pgAdmin4 website

2020-08-25 Thread Dave Page
Thanks, patch applied. Can you please make one more tweak though (obviously against the current git head) - specifically, the page titles should be more specific, e.g. Styleguide: Iconography Styleguide: Color Palette etc. This should be reflected in the title tag of the pages as well as the he

Re: [pgAdmin][RM5990]: Update the style guide on pgAdmin4 website

2020-08-20 Thread Dave Page
Hi On Wed, Aug 19, 2020 at 10:20 AM Nikhil Mohite < nikhil.moh...@enterprisedb.com> wrote: > Hi Dave/Team, > > As per discussion with the UX team, Few updates on style-guide patch. > 1. I have missed removing the "proposed" keyword from the "High-Contract" > theme name. > Right. > 2. I have no

Re: [pgAdmin][RM5990]: Update the style guide on pgAdmin4 website

2020-08-19 Thread Nikhil Mohite
Hi Dave/Team, As per discussion with the UX team, Few updates on style-guide patch. 1. I have missed removing the "proposed" keyword from the "High-Contract" theme name. 2. I have not added the "Writing Style" section in the style guide as we don’t have the information for it (UX design is also pe