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

Add shortcut link to read the docs within app navbar #576

Closed
wants to merge 3 commits into from
Closed

Conversation

Raruto
Copy link
Collaborator

@Raruto Raruto commented Feb 14, 2024

Motivation

Have a quick reference to the current code documentation, eg: https://g3w-suite.readthedocs.io/en/v3.7.x/

Before

image

After

image

@Raruto Raruto added the ux Changes to a current behavior to improve user experience label Feb 14, 2024
@Raruto Raruto added this to the v3.10 milestone Feb 14, 2024
@Raruto Raruto requested a review from volterra79 February 14, 2024 11:11
@volterra79 volterra79 requested a review from leolami February 14, 2024 13:17
@leolami
Copy link
Member

leolami commented Feb 14, 2024

Hi @Raruto, thanks for the suggestion but I don't think it's useful for the following reason

All customers would have the web client with a Help button pointing to the complete suite manual with aspects that the user is not interested in

You have to think about the use of the client by an low leverl user and not by an administrator

It could be useful if the associated link could be customized with links to their specific manuals

But in this case the button should only be visible if the link has been defined

@leolami
Copy link
Member

leolami commented Feb 14, 2024

Maybe that button is more useful in the ADMIN session

@volterra79 volterra79 marked this pull request as draft March 4, 2024 13:34
@Raruto Raruto modified the milestones: v3.10, v3.11 Mar 25, 2024
@Raruto Raruto removed this from the v3.11 milestone Jul 26, 2024
# Conflicts:
#	src/components/App.vue
@Raruto
Copy link
Collaborator Author

Raruto commented Nov 14, 2024

Superseded by: #679

@Raruto Raruto closed this Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ux Changes to a current behavior to improve user experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants