Opened 4 weeks ago
Last modified 4 weeks ago
#36097 assigned Cleanup/optimization
Remove the geographic database functions summary table
Reported by: | Sarah Boyce | Owned by: | Sarah Boyce |
---|---|---|---|
Component: | GIS | Version: | 5.1 |
Severity: | Normal | Keywords: | |
Cc: | Triage Stage: | Accepted | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Pull Requests: | 19044 | ||
Description (last modified by ) ¶
The Geographic database function summary table is malformed and slightly misleading (rows make it seem like all functions relate to each other)
It is also fairly difficult to update.
I believe this table is there to group the functions. Instead of a table I believe we should group the functions with headers
https://docs.djangoproject.com/en/5.1/ref/contrib/gis/functions/
According to the ticket's flags, the next step(s) to move this issue forward are:
- For anyone except the patch author to review the patch using the patch review checklist and either mark the ticket as "Ready for checkin" if everything looks good, or leave comments for improvement and mark the ticket as "Patch needs improvement".
Change History (8)
by , 4 weeks ago
Attachment: | image-20250114-111030.png added |
---|
comment:1 by , 4 weeks ago
Description: | modified (diff) |
---|
comment:2 by , 4 weeks ago
Has patch: | set |
---|
comment:3 by , 4 weeks ago
by , 4 weeks ago
Attachment: | image-20250114-121804.png added |
---|
by , 4 weeks ago
Attachment: | image-20250114-121855.png added |
---|
Note:
See TracTickets
for help on using tickets.
I think that the summary table of the geographic functions of the database is a great-added value to the page, giving an immediate picture of the content of the page.
But I also think that the new hierarchical organization of the headers of the geographic functions of the database is a great improvement.
Before giving an opinion in favor of its removal of the table, could you post a screenshot of the contents' column with the new headers?