This repository has been archived by the owner. It is now read-only.

Form/restore select doc #360

Closed
wants to merge 3 commits into
from

Conversation

Projects
None yet
3 participants
@bakura10
Contributor

bakura10 commented Oct 4, 2012

No description provided.

-
- Get the label for the empty option (null if none).
-
- :rtype: string

This comment has been minimized.

@weierophinney

weierophinney Oct 4, 2012

Member

Why are you removing the above? This should stay in.

@weierophinney

weierophinney Oct 4, 2012

Member

Why are you removing the above? This should stay in.

This comment has been minimized.

@bakura10

bakura10 Oct 5, 2012

Contributor

It's not removed if you check at the raw file... I don't understand what GitHub does here..,

Envoyé de mon iPhone

Le 4 oct. 2012 à 23:07, Matthew Weier O'Phinney notifications@github.com a écrit :

In docs/languages/en/modules/zend.form.element.select.rst:

- Return the value options.

- :rtype: array

-.. function:: setEmptyOption($emptyOption)

- :noindex:

- Optionally set a label for an empty option (option with no value). It is set to "null" by default, which means that no empty option will be rendered.

-.. function:: setEmptyOption()

- :noindex:

- Get the label for the empty option (null if none).

  • :rtype: string
    Why are you removing the above? This should stay in.


Reply to this email directly or view it on GitHub.

@bakura10

bakura10 Oct 5, 2012

Contributor

It's not removed if you check at the raw file... I don't understand what GitHub does here..,

Envoyé de mon iPhone

Le 4 oct. 2012 à 23:07, Matthew Weier O'Phinney notifications@github.com a écrit :

In docs/languages/en/modules/zend.form.element.select.rst:

- Return the value options.

- :rtype: array

-.. function:: setEmptyOption($emptyOption)

- :noindex:

- Optionally set a label for an empty option (option with no value). It is set to "null" by default, which means that no empty option will be rendered.

-.. function:: setEmptyOption()

- :noindex:

- Get the label for the empty option (null if none).

  • :rtype: string
    Why are you removing the above? This should stay in.


Reply to this email directly or view it on GitHub.

This comment has been minimized.

@bakura10

bakura10 Oct 5, 2012

Contributor

This comment has been minimized.

@weierophinney

weierophinney Oct 5, 2012

Member

Very odd! But yeah, definitely still there!

@weierophinney

weierophinney Oct 5, 2012

Member

Very odd! But yeah, definitely still there!

@uncle-fred

This comment has been minimized.

Show comment
Hide comment
@uncle-fred

uncle-fred Oct 4, 2012

Contributor

I proposed some write-up for the Select element some days ago and I think Matthew even merged it. But I must say, your version is way better. So, I don’t mind you overwriting it :) Looks great! Great job.

Contributor

uncle-fred commented Oct 4, 2012

I proposed some write-up for the Select element some days ago and I think Matthew even merged it. But I must say, your version is way better. So, I don’t mind you overwriting it :) Looks great! Great job.

weierophinney added a commit that referenced this pull request Oct 5, 2012

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.