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

Server checkbox tooltip #490

Merged
merged 1 commit into from
Nov 22, 2024
Merged

Conversation

Xpirix
Copy link
Collaborator

@Xpirix Xpirix commented Nov 22, 2024

Fix for #487

image

Copy link
Contributor

@Gustry Gustry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM thanks, it's indeed better for the UX :)

db_index=True,
help_text=_(
"A server plugin is a plugin which can run on QGIS Server,"
" by having a entrypoint <b>serverClassFactory</b>, see the"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
" by having a entrypoint <b>serverClassFactory</b>, see the"
" by having a entrypoint <pre>serverClassFactory</pre>, see the"

@Xpirix Xpirix merged commit 1fde67f into qgis:master Nov 22, 2024
2 checks passed
@Gustry
Copy link
Contributor

Gustry commented Nov 22, 2024

Sorry, you don't prefer <pre> instead of <b> ? It find it better for computing languages ;-)

@Xpirix
Copy link
Collaborator Author

Xpirix commented Nov 22, 2024

Oops, I'm really sorry. I missed your suggestion; I will fix it right away.

@Xpirix
Copy link
Collaborator Author

Xpirix commented Nov 22, 2024

I've applied it with a new PR at #492

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants