-
Notifications
You must be signed in to change notification settings - Fork 48
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
Organize admin views #3841
Organize admin views #3841
Conversation
⛔ Feature branch deployment currently inactive.If the PR is still open, you can add the |
8fb4329
to
1501a14
Compare
@ecamp/core what do you think of this draft? |
I like it! Minor:
|
</v-navigation-drawer> | ||
</template> | ||
|
||
<script> | ||
export default { | ||
name: 'SideBar', | ||
props: { | ||
// TODO: Implement title in SideBarProgram as well |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is with this todo?
either do it and delete, or create an issue and delete please
d4b32e4
to
e0bb34f
Compare
e0bb34f
to
47b5a1f
Compare
Navigate to the list of materiallists in the NavBottombar. Add new view for the list of materiallists to navigate to the details of one materiallist. Rename the views/admin/(,Admin)MaterialLists component to avoid name clash with the new component. Add quick way to differentiate the target routes in the materialListRoute helper for the NavBottombar and NavTopbar. May have been introduced in ecamp#3841. Also add guard if camp is not loaded to materialListRoute.
The goal of this PR is to reorganize the admin views. Also provide better wording for non-managers on the admin/info page. This will make the experience more consistent across devices.
Member view:
![Admin page with TopNavbarItem "Infos"](https://private-user-images.githubusercontent.com/3001985/270313052-01907f92-4da2-420a-8e46-7a3ae7a3b544.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjAyNTcwMzcsIm5iZiI6MTcyMDI1NjczNywicGF0aCI6Ii8zMDAxOTg1LzI3MDMxMzA1Mi0wMTkwN2Y5Mi00ZGEyLTQyMGEtOGU0Ni03YTNhZTdhM2I1NDQucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDcwNiUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDA3MDZUMDkwNTM3WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9MjRjNjNlNDEyYjNkMDI4ODZiMWM2ZTAwOWIxMzQ1NWQxYjY0MmM5OWVmMDRjMjRhODgwMzYwYzZjZDc5YTMzNCZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.kcLLjbuk5rAGx1YVI722wSyWNm00lMzBDorRMtvyy2c)
Manager view:
![Admin page with TopNavbarItem "Admin"](https://private-user-images.githubusercontent.com/3001985/270313166-a682316e-8461-4e33-a22c-cb2726cc6ec3.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjAyNTcwMzcsIm5iZiI6MTcyMDI1NjczNywicGF0aCI6Ii8zMDAxOTg1LzI3MDMxMzE2Ni1hNjgyMzE2ZS04NDYxLTRlMzMtYTIyYy1jYjI3MjZjYzZlYzMucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDcwNiUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDA3MDZUMDkwNTM3WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9OWY2NjgxMDlkNDdhZTRmZTk5MjhlNGI1ZTExYThhNTBlMzZkYmNmMDJhMjZjOGExNWJiYTI2MDBjYmQ3ODRlYyZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.clTP3Tjzv4bAUvyuxXGkAricdb0OTvr7UQXZWEHvylk)
In a next PR, I would like to change the api-input prop
![Bildschirmfoto 2023-09-25 um 12 59 20](https://private-user-images.githubusercontent.com/3001985/270319991-17ee7764-ea82-4169-980b-10cbae9d6521.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjAyNTcwMzcsIm5iZiI6MTcyMDI1NjczNywicGF0aCI6Ii8zMDAxOTg1LzI3MDMxOTk5MS0xN2VlNzc2NC1lYTgyLTQxNjktOTgwYi0xMGNiYWU5ZDY1MjEucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDcwNiUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDA3MDZUMDkwNTM3WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9M2Y1OTQ0MDAzZTdiODZjY2M5ZjViMjc5OWY4ZmI4NmYyNzk0Y2MxYmZiNjk3N2MxYmM3Nzk2NmQ2N2MxMGYyZCZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.39SY6X2SduChoHfrChRquHgWZJxOqU8wpON86T3KwN8)
disabled
toreadonly
and remove the outlined style of the api-input if it is readonly.FYI: Material lists would be a new view with renaming capabilities. This was not implemented in this draft as it is blocked by #3797.