Hi Dave, On Fri, Feb 21, 2025 at 3:21 PM Dave Page <dp...@pgadmin.org> wrote:
> > > On Fri, 21 Feb 2025 at 09:38, Aditya Toshniwal < > aditya.toshni...@enterprisedb.com> wrote: > >> Hi Hackers, >> >> I've been working on a new feature to restore a plain SQL using the >> pgAdmin restore dialog. Internally, it will select pg_restore or psql based >> on the format. >> >> Now, when restoring plain format the tabs like Data Options, >> Query Options are not applicable. So instead of going and disabling all the >> fields under these tabs, I think we can simply disable the complete tab. >> >> I've come up with the following UI design for the disabled tabs. The tabs >> will enable/disable based on the format selected. >> >> Please share your feedback on this. >> >> >> [image: image.png] >> > > Normally when something is disabled, it is simply greyed out and made to > not respond to input. Why not do the same here instead of designing a new > way of indicating disabled state? > I thought of adding an icon for better accessibility. Greying is pretty straightforward. > > -- > Dave Page > pgAdmin: https://www.pgadmin.org > PostgreSQL: https://www.postgresql.org > pgEdge: https://www.pgedge.com > > -- Thanks, Aditya Toshniwal pgAdmin Hacker | Sr. Staff SDE II | *enterprisedb.com* <https://www.enterprisedb.com/> "Don't Complain about Heat, Plant a TREE"
