Обсуждение: [pgAdmin4][Patch]: RM 1729 Role privileges are badly displayed

Поиск
Список
Период
Сортировка

[pgAdmin4][Patch]: RM 1729 Role privileges are badly displayed

От
Akshay Joshi
Дата:
Hi All

Please find the attached patch to fix the RM #1729 Role privileges are badly displayed

Please review it.

--
Akshay Joshi
Principal Software Engineer 


Phone: +91 20-3058-9517
Mobile: +91 976-788-8246
Вложения

Re: [pgAdmin4][Patch]: RM 1729 Role privileges are badly displayed

От
Dave Page
Дата:


On Fri, Oct 28, 2016 at 10:23 AM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi All

Please find the attached patch to fix the RM #1729 Role privileges are badly displayed

Please review it.

Hi

This improves the display on the properties dialogue, though it could still make much better use of the space (for example, the switches could almost be right-justified). However, it messes up the Properties pane on the browser, as it shows the switches indented compared to other controls.

Thanks. 

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Re: [pgAdmin4][Patch]: RM 1729 Role privileges are badly displayed

От
Akshay Joshi
Дата:
Hi All 

On Fri, Oct 28, 2016 at 8:53 PM, Dave Page <dpage@pgadmin.org> wrote:


On Fri, Oct 28, 2016 at 10:23 AM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi All

Please find the attached patch to fix the RM #1729 Role privileges are badly displayed

Please review it.

Hi

This improves the display on the properties dialogue, though it could still make much better use of the space (for example, the switches could almost be right-justified). However, it messes up the Properties pane on the browser, as it shows the switches indented compared to other controls.

   Attached is the modified patch as per Dave's suggestion. Please review it.  

Thanks. 

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



--
Akshay Joshi
Principal Software Engineer 


Phone: +91 20-3058-9517
Mobile: +91 976-788-8246
Вложения

Re: [pgAdmin4][Patch]: RM 1729 Role privileges are badly displayed

От
Dave Page
Дата:
Thanks, patch applied.

On Tuesday, November 1, 2016, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi All 

On Fri, Oct 28, 2016 at 8:53 PM, Dave Page <dpage@pgadmin.org> wrote:


On Fri, Oct 28, 2016 at 10:23 AM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi All

Please find the attached patch to fix the RM #1729 Role privileges are badly displayed

Please review it.

Hi

This improves the display on the properties dialogue, though it could still make much better use of the space (for example, the switches could almost be right-justified). However, it messes up the Properties pane on the browser, as it shows the switches indented compared to other controls.

   Attached is the modified patch as per Dave's suggestion. Please review it.  

Thanks. 

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



--
Akshay Joshi
Principal Software Engineer 


Phone: +91 20-3058-9517
Mobile: +91 976-788-8246


--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company