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

Fixed #203 #255

Merged
merged 1 commit into from May 21, 2015
Merged

Fixed #203 #255

merged 1 commit into from May 21, 2015

Conversation

Defman21
Copy link
Contributor

@Defman21 Defman21 commented May 6, 2015

I sure as hell my changes requires some refactoring but my English not that good to call variables better than now.

@Defman21
Copy link
Contributor Author

Defman21 commented May 6, 2015

<?php

/**
 * Do something
 * @param Boolean $arg1 If true - do some stuff
 * @param Boolean $arg2
 * @return Boolean
 * Another docs
 * @since 1.0
 * @author John Smith jsmith@pod1.mars
 * @see http://google.com/
 */
function test($arg1 = true, $arg2 = false) {
  return true;
}

test(|); // | - cursor

The tooltip will be

boolean test(Boolean $arg1 = true, Boolean $arg2 = false)
Do something
<boolean>$arg1 If True - do some stuff
<boolean>$arg2
Returns boolean
Another docs
Since 1.0
Author John Smith jsmith@pod1.mars
See http://google.com/

@Naatan
Copy link
Member

Naatan commented May 12, 2015

Nice work Defman! I'll try to review next week and merge it in if all is good.

Naatan added a commit that referenced this pull request May 21, 2015
CodeIntel: Improved legibility of PHPdoc tooltips (contributed by Defman) - resolves #203
rn=
@Naatan Naatan merged commit 5e72ba6 into Komodo:master May 21, 2015
@Naatan
Copy link
Member

Naatan commented May 21, 2015

Integrated! Thanks Defman :) Made some minor alterations here - b832360

@Defman21
Copy link
Contributor Author

Thank you for your fixes :)

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.

None yet

2 participants