Re: Result grid rendering (was: Re: [pgAdmin4][Pattch] - RM #3673 -"Download as .csv" F8 does NOT work when one of joined files is a TEMPORARY file)

Поиск
Список
Период
Сортировка
От Dave Page
Тема Re: Result grid rendering (was: Re: [pgAdmin4][Pattch] - RM #3673 -"Download as .csv" F8 does NOT work when one of joined files is a TEMPORARY file)
Дата
Msg-id CA+OCxoyk7L879FUk7zHd4kFhZueXg6nB+_hgv=21YL4Dztg++A@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Result grid rendering (was: Re: [pgAdmin4][Pattch] - RM #3673 -"Download as .csv" F8 does NOT work when one of joined files is a TEMPORARY file)  (Khushboo Vashi <khushboo.vashi@enterprisedb.com>)
Ответы Re: Result grid rendering (was: Re: [pgAdmin4][Pattch] - RM #3673 - "Download as .csv" F8 does NOT work when one of joined files is a TEMPORARY file)  (Khushboo Vashi <khushboo.vashi@enterprisedb.com>)
Список pgadmin-hackers
Дерево обсуждения


On Fri, Mar 1, 2019 at 5:39 AM Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi Dave,

Please find the attached fix.

Thanks,
Khushboo


On Thu, Feb 28, 2019 at 6:58 PM Dave Page <dpage@pgadmin.org> wrote:
Hi

On Thu, Feb 28, 2019 at 3:46 AM Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi Dave,

On Wed, Feb 27, 2019 at 4:30 PM Dave Page <dpage@pgadmin.org> wrote:
Hi

Thanks - that fixed the double scrollbar, but not the other issue. Any ideas on that one?
Can you please provide the exact steps to reproduce the second issue as after fixing the first one I can't reproduce it. 

Sure - movie seems easiest, so grab some popcorn and put the attached on the big screen! 
Unfortunately, I don't have a big screen, so I missed the opportunity to have popcorn. :)

Booo :-(

Well, anyway - the sizing issue seems OK now, but if I have a table (id serial primary key, data text) with enough rows that I need to scroll, when I scroll to the end I can edit the id field in place, but the text editor popup doesn't show up. If I try to edit a row further up, it seems to appear in a weird place, so I suspect it's just rendering off-screen.

I think we should render it directly against the cell - above if the cell is >50% down in the overall screen, else above, and something similarly sane for the left/right.

Not sure if this is a regression or not, but it's unusable as-is, so can you fix that too? I'll commit your current patch.

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

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

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: Murtuza Zabuawala
Дата:
Сообщение: Re: [RM#3998] Custom encode forward slashes in the grid title
Следующее
От: Dave Page
Дата:
Сообщение: pgAdmin 4 commit: Fix resizing of the results grid, when full of data.