-
-
Notifications
You must be signed in to change notification settings - Fork 597
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
help text and link for form labels #138 #143
Conversation
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.
Where are the tooltips? I didn't find them.
I can‘t remember the status of the branch. But I won‘t merge it anyways, as I am working on updating to Symfony 4.1 in #160 which will bring „help“ labels out of the box. |
$view->vars['documentation'] = $options['documentation'] ?? ''; | ||
} | ||
|
||
public function configureOptions(OptionsResolver $resolver) |
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.
do we want to include docheader everywhere or not?
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.
no, the documentation link will only be displayed where it has an value.
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.
I think you misunderstood me. I mean the code documentation above the method.
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.
Yes I did indeed ;-) I'll update the PR later, thanks for the check!
We need to improve the Scrutinizer and phpcs style checks, see #158 to find such stuff automatically.
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.
added the phpdoc
Okay, the branch is done. You can test if you want.
with
to see it. This form has also the new "link to docu icon" besides the form title. |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Description
Fixes #138 by adding methods to:
Added docu about formats that can be entered in the
duration
field - the link is in the edit timesheet form next to the title (the ? icon).Symfony supports "help" text out-of-the-box with the recently executed upgrade to 4.1 for single fields and I checked it => will be displayed.