Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

convert admin maps page to modals #1197

Open
huss opened this issue Apr 4, 2024 · 4 comments · May be fixed by #1314
Open

convert admin maps page to modals #1197

huss opened this issue Apr 4, 2024 · 4 comments · May be fixed by #1314
Labels
p-medium-priority reserved Expected to be assigned to developer by project so others should not work on without prior approval. t-enhancement This issues tracks a potential improvement to the software
Milestone

Comments

@huss
Copy link
Member

huss commented Apr 4, 2024

Is your feature request related to a problem? Please describe.

The admin maps page does not conform to the new modal look in OED.

Describe the solution you'd like

The look of this page will change substantially to align with the new modal card look in OED. The top of the page will have a create button that takes the admin to the page to create a map that uses the current pages. Note this is actually a sequence of pages. A card for each map will have the map name, map display & note (30 characters max as on current modal pages) listed along with a button to edit the map.

When the edit button is clicked, a modal pops up that shows all the values on the current page except the id that will not be shown. The admin can modify any of them except the last modify date that is set by the software. The actions taken on a change will be similar to the current page but the look should be consistent with the new modal pages. For example, the button to delete the map is similar to the conversion page.

Note the current page allows for editing multiple maps but that will not be allowed on the new page. Only one map at a time to be consistent with other modal pages. The change to the modal setup may change a number of the functions and state interactions.

Given what is done when a map is created/calibrated, it is unclear if that should be a modal as we want it to be the full size of the page. At some point, it should be checked to see if changing it would make sense.

Describe alternatives you've considered

Leave as is

Additional context

This replaces issue #891 since the hooks was replaced with issue #1145 and the modal part is here.

@huss huss added t-enhancement This issues tracks a potential improvement to the software p-medium-priority labels Apr 4, 2024
@huss huss added this to the 1.1 release milestone Apr 4, 2024
@huss huss added the reserved Expected to be assigned to developer by project so others should not work on without prior approval. label May 31, 2024
@ChrisMart21
Copy link
Contributor

@huss is this issue still reserved? I'm currently working on many of the admin pages maps included. This rework changes how admin pages update edits locally using redux state. The pages will also share a modal across all admin pages. It seems that some of the work I'm doing overlaps with this issue.
This is a screen cap of what I've got going so far
image

@huss
Copy link
Member Author

huss commented Jun 11, 2024

@ChrisMart21 There is a team working on the admin and CSV pages. If you are already working on admin map then that is fine. If they have more time they can address other issues. Do you want me to assign you this issue?

Also, are your updates to edit Redux state just for maps or will it impact all admin pages? If so, will they likely cause conflicts with the team working on some of the pages?

@ChrisMart21
Copy link
Contributor

@huss Unfortunately I've touched all admin pages quite a bit. I'm unsure if this will cause issues for the other teams, but definitely changes it quite a bit. I don't want to step on any toes, or interfere with other teams... could we sync up to discuss this?

@huss
Copy link
Member Author

huss commented Jun 11, 2024

@ChrisMart21 Okay. I think it will be okay. We can talk privately about this.

@huss huss linked a pull request Jul 23, 2024 that will close this issue
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p-medium-priority reserved Expected to be assigned to developer by project so others should not work on without prior approval. t-enhancement This issues tracks a potential improvement to the software
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants