Revert r74387: it may not fix the IE8 jumping bug in all cases, but it definitely...

diff --git a/includes/EditPage.php b/includes/EditPage.php
index e8bfd95..a79e272 100644 (file)
--- a/includes/EditPage.php
+++ b/includes/EditPage.php
@@ -371,6 +371,8 @@ class EditPage {
                if ( $wgUser->getOption( 'uselivepreview', false ) ) {
                        $wgOut->addModules( 'mediawiki.legacy.preview' );
                }
+               // Bug #19334: textarea jumps when editing articles in IE8
+               $wgOut->addStyle( 'common/IE80Fixes.css', 'screen', 'IE 8' );
 
                $permErrors = $this->getEditPermissionErrors();
                if ( $permErrors ) {

diff --git a/skins/common/IE80Fixes.css b/skins/common/IE80Fixes.css
new file mode 100644 (file)
index 0000000..b6360f6
--- /dev/null
+++ b/skins/common/IE80Fixes.css
@@ -0,0 +1,15 @@
+/**
+ * Fixes textarea scrolling bug (bug #19334).  The bug only occurs when a
+ * percentage width is given, so instead of width: 100%, use min-width: 100%;
+ * max-width: 100%.  We also need to give a fixed width for the actual width
+ * property for the hack to work, although the actual value (500px here) ends
+ * up being ignored; min/max-width take precedence.
+ *
+ * More info: http://grantovich.net/posts/2009/06/that-weird-ie8-textarea-bug/
+ */
+#wpTextbox1 {
+       height: 390px;
+       width: 500px;
+       min-width: 100%;
+       max-width: 100%;
+}