Skip to content

Commit f044f59

Browse files
committed
Merge pull request #212 from magento-mpi/prs
[Github] Merge public Github commits
2 parents b20ef1c + 43a4367 commit f044f59

File tree

2 files changed

+33
-6
lines changed

2 files changed

+33
-6
lines changed

README.md

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ Use the following table to verify you have the correct prerequisites to install
2727
</tr>
2828
<tr>
2929
<td>Apache 2.2 or 2.4</td>
30-
<td>Ubuntu: <code>apache -v</code><br>
30+
<td>Ubuntu: <code>apache2 -v</code><br>
3131
CentOS: <code>httpd -v</code></td>
3232
<td><a href="http://devdocs.magento.com/guides/v1.0/install-gde/prereq/apache.html">Apache</a></td>
3333
</tr>
@@ -57,3 +57,17 @@ After verifying your prerequisites, perform the following tasks in order to prep
5757
* <a href="http://devdocs.magento.com/guides/v1.0/install-gde/install/install-web.html">Install Magento software using the web interface</a>
5858
* <a href="http://devdocs.magento.com/guides/v1.0/install-gde/install/install-cli.html">Install Magento software using the command line</a>
5959
2. <a href="http://devdocs.magento.com/guides/v1.0/install-gde/install/verify.html">Verify the installation</a>
60+
61+
<h2>Contributing to the Magento 2 code base</h2>
62+
Contributions can take the form of new components or features, changes to existing features, tests, documentation (such as developer guides, user guides, examples, or specifications), bug fixes, optimizations, or just good suggestions.
63+
64+
To make learn about how to make a contribution, click [here][1].
65+
66+
To learn about issues, click [here][2]. To open an issue, click [here][3].
67+
68+
To suggest documentation improvements, click [here][4].
69+
70+
[1]: <http://devdocs.magento.com/guides/v1.0/contributor-guide/CONTRIBUTING.html>
71+
[2]: <http://devdocs.magento.com/guides/v1.0/contributor-guide/CONTRIBUTING.html#report>
72+
[3]: <https://github.com/magento/magento2/issues>
73+
[4]: <http://devdocs.magento.com>

lib/internal/Magento/Framework/Data/Form/Element/Textarea.php

Lines changed: 18 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,16 @@
1515

1616
class Textarea extends AbstractElement
1717
{
18+
/**
19+
* Default number of rows
20+
*/
21+
const DEFAULT_ROWS = 2;
22+
23+
/**
24+
* Default number of columns
25+
*/
26+
const DEFAULT_COLS = 15;
27+
1828
/**
1929
* @param Factory $factoryElement
2030
* @param CollectionFactory $factoryCollection
@@ -30,8 +40,12 @@ public function __construct(
3040
parent::__construct($factoryElement, $factoryCollection, $escaper, $data);
3141
$this->setType('textarea');
3242
$this->setExtType('textarea');
33-
$this->setRows(2);
34-
$this->setCols(15);
43+
if (!$this->getRows()) {
44+
$this->setRows(self::DEFAULT_ROWS);
45+
}
46+
if (!$this->getCols()) {
47+
$this->setCols(self::DEFAULT_COLS);
48+
}
3549
}
3650

3751
/**
@@ -65,9 +79,8 @@ public function getHtmlAttributes()
6579
public function getElementHtml()
6680
{
6781
$this->addClass('textarea');
68-
$html = '<textarea id="' . $this->getHtmlId() . '" name="' . $this->getName() . '" ' . $this->serialize(
69-
$this->getHtmlAttributes()
70-
) . $this->_getUiId() . ' >';
82+
$html = '<textarea id="' . $this->getHtmlId() . '" name="' . $this->getName() . '" '
83+
. $this->serialize($this->getHtmlAttributes()) . $this->_getUiId() . ' >';
7184
$html .= $this->getEscapedValue();
7285
$html .= "</textarea>";
7386
$html .= $this->getAfterElementHtml();

0 commit comments

Comments
 (0)