Last modified: 2013-04-23 19:34:24 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T49396, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 47396 - [Regression] VisualEditor: Edit tab should have tooltip and accesskey
[Regression] VisualEditor: Edit tab should have tooltip and accesskey
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
MediaWiki integration (Other open bugs)
unspecified
All All
: High normal
: VE-deploy-2013-04-29
Assigned To: Krinkle
: code-update-regression
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-04-19 07:10 UTC by Erik Moeller
Modified: 2013-04-23 19:34 UTC (History)
5 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Erik Moeller 2013-04-19 07:10:36 UTC
Ordinarily there's an accesskey (e.g. alt+shift+E in Firefox) on the "Edit" tab in Vector. The accesskey is made known to the user through a tooltip. VisualEditor eliminates this accesskey and tooltip, and adds a new "Edit source" tab without an accesskey or tooltip. 

Ideally both the "Edit" and "Edit source" tab should have accesskeys and tooltips to allow quick switching between the modes for power users.
Comment 1 Krinkle 2013-04-19 09:02:15 UTC
When mw.ViewPageTarget has its tabLayout in "replace" mode (where it replaces the regular Edit tab with a VE-version and adds an Edit source tab) the code explicitly copies over the label, tooltip and accesskey.

It appears this isn't working anymore. I worked a week ago. The code hasn't changed so it looks like something else has broken it.

Moving bug to Integration (perhaps the component should be renamed to "MediaWiki integration")
Comment 2 Gerrit Notification Bot 2013-04-22 08:02:08 UTC
Related URL: https://gerrit.wikimedia.org/r/60241 (Gerrit Change If598552fac639da645a8b1273c5fc6028695fcc1)
Comment 3 Krinkle 2013-04-22 08:07:41 UTC
Change-Id: If598552fac639da645a8b1273c5fc6028695fcc1

> @@ -777,10 +778,10 @@ ve.init.mw.ViewPageTarget.prototype.setupSkinTabs
>  			this.veEditUri,
> -			$caEdit.text(),
> +			$caEditA.text(),
>  			$caEdit.attr( 'id' ),
> -			$caEdit.attr( 'title' ),
> -			$caEdit.attr( 'accesskey' ),
> +			$caEditA.attr( 'title' ),
> +			$caEditA.attr( 'accesskey' ),
>  			caVeEditSource
>  		);
>  	}
Comment 4 James Forrester 2013-04-23 19:34:24 UTC
Code is now merged and will go out on Thursday.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links