Re: [pgAdmin4][patch]: Feature #1407 - Support map view for PostGISquery result sets

Поиск
Список
Период
Сортировка
От Dave Page
Тема Re: [pgAdmin4][patch]: Feature #1407 - Support map view for PostGISquery result sets
Дата
Msg-id CA+OCxowWNbLygV7uejDRWMAhwsL9+2TtPtNcQEec4M=u-hTitg@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [pgAdmin4][patch]: Feature #1407 - Support map view for PostGISquery result sets  (Xuri Gong <xurigoong@gmail.com>)
Ответы Re: [pgAdmin4][patch]: Feature #1407 - Support map view for PostGIS query result sets  (Xuri Gong <xurigoong@gmail.com>)
Список pgadmin-hackers
Дерево обсуждения
Hi

On Sat, Aug 4, 2018 at 1:27 PM, Xuri Gong <xurigoong@gmail.com> wrote:
Hi,

I have added documents for geometry viewer. Please review this newer patch.

For test you can install PostGIS and import this database[1].

Thank you!


I finally got a chance to play with this! It's pretty awesome - I think some tweaks are needed to polish it off though:

1) The column headers should be resized to take into account the button size. It looks a little weird when they obscure the text.

2) It would be good to be able to select a subset of rows. The grid already supports this... I'd suggest that the header button should display only the row(s) that are currently selected. That would also allow the removal of the per-row buttons.

3) Can the panel be made into a regular dialogue, with a close button and ability to resize and move it?

I haven't looked at the code in any great depth due to time constraints, but I didn't see anything particularly untoward except some of the formatting; for example:

if(command === 'view-all-geometries'){
...
  }else{
Should look more like:

if (command === 'view-all-geometries') {
...
  } else {
Thanks - that's good work!
 

Xuri Gong <xurigoong@gmail.com> 于2018年7月31日周二 下午4:47写道：
Hi,

This is the patch for viewing geometry data in pgAdmin4. I have implemented it as follows:
  • Add cell button and column header button in datagrid for geometry and geography type data. [screenshot 1]
  • When user clicks the 'view' button, it parses the data and opens a map to show the geometry. [screenshot 2, 3, 4]
  • When user clicks the geometry in the map, it shows the properties. [screenshot 5]
I have created unit test for this. I have also created a test database[1] including geometries of different types to run test manually. Please review it. Thank you.

Regards,
Xuri Gong




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

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

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

Предыдущее
От: Dave Page
Дата:
Сообщение: Re: [pgAdmin4][Patch]: RM #3407 Fix layout of keyboard shortcutpreferences panels
Следующее
От: Akshay Joshi
Дата:
Сообщение: [pgAdmin4][Patch]: RM #3528 Query Tool doesn't handle 'Too manyconnection' error properly