r69337 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r69336‎ | r69337 | r69338 >
Date:18:50, 14 July 2010
Author:simetrical
Status:reverted (Comments)
Tags:
Comment:
Remove 'minordefault' preference completely

See bug 24313. The preference encourages people to mark changes minor
by mistake, but it's better to have minor changes not marked minor than
non-minor changes marked minor. enwiki users at [[Help talk:Minor
edit]] seemed to think it was a good idea. Plus, I'm always in favor of
removing user preferences where possible.

The only likely fallout of note is some bot operators who didn't notice
this suddenly find their bot not marking anything minor. I doubt it
will cause major problems, though.
Modified paths:
  • /trunk/phase3/RELEASE-NOTES (modified) (history)
  • /trunk/phase3/includes/DefaultSettings.php (modified) (history)
  • /trunk/phase3/includes/EditPage.php (modified) (history)
  • /trunk/phase3/includes/Preferences.php (modified) (history)
  • /trunk/phase3/includes/User.php (modified) (history)
  • /trunk/phase3/includes/api/ApiEditPage.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesAce.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesAf.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesAln.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesAm.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesAn.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesAng.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesAr.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesArz.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesAs.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesAst.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesAvk.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesAy.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesAz.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesBa.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesBar.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesBat_smg.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesBcc.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesBcl.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesBe.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesBe_tarask.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesBg.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesBn.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesBpy.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesBqi.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesBr.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesBs.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesCa.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesCe.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesCeb.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesCh.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesCkb_arab.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesCps.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesCrh_cyrl.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesCrh_latn.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesCs.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesCsb.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesCv.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesCy.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesDa.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesDe.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesDiq.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesDsb.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesEl.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesEn.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesEo.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesEs.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesEt.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesEu.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesExt.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesFa.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesFi.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesFo.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesFr.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesFrc.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesFrp.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesFrr.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesFur.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesFy.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesGa.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesGag.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesGan_hans.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesGan_hant.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesGd.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesGl.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesGrc.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesGsw.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesGu.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesGv.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesHa.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesHak.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesHe.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesHi.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesHif_latn.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesHr.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesHsb.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesHt.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesHu.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesHy.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesIa.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesId.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesIg.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesIlo.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesInh.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesIo.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesIs.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesIt.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesJa.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesJut.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesJv.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesKa.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesKaa.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesKab.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesKiu.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesKk_arab.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesKk_cyrl.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesKk_latn.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesKm.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesKn.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesKo.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesKrc.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesKsh.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesKu_latn.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesLa.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesLad.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesLb.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesLfn.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesLi.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesLmo.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesLo.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesLt.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesLv.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesLzh.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesMdf.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesMg.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesMhr.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesMk.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesMl.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesMn.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesMr.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesMrj.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesMs.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesMt.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesMwl.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesMyv.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesMzn.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesNah.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesNan.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesNds.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesNds_nl.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesNe.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesNew.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesNl.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesNn.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesNo.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesNso.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesOc.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesOs.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesPam.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesPl.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesPms.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesPrg.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesPt.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesPt_br.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesQqq.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesQu.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesRgn.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesRo.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesRoa_tara.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesRu.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesRue.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesSa.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesSah.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesSc.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesScn.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesSco.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesSdc.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesSe.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesSei.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesSh.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesShi.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesSi.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesSk.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesSl.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesSli.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesSo.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesSq.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesSr_ec.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesSr_el.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesSrn.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesStq.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesSu.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesSv.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesSw.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesSzl.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesTa.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesTcy.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesTe.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesTg_cyrl.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesTg_latn.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesTh.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesTk.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesTl.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesTpi.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesTr.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesTt_cyrl.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesUg_arab.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesUk.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesUr.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesVec.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesVep.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesVi.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesVmf.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesVo.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesVro.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesWa.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesWar.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesWo.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesWuu.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesXal.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesYi.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesYo.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesYue.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesZea.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesZh_hans.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesZh_hant.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesZh_tw.php (modified) (history)
  • /trunk/phase3/maintenance/language/messages.inc (modified) (history)

Follow-up revisions

RevisionCommit summaryAuthorDate
r75815Revert r69337 (Removal of minordefault preference)overlordq18:58, 1 November 2010

Comments

#Comment by Simetrical (talk | contribs)   18:53, 14 July 2010

Discussed in #mediawiki beforehand, no objections. Marking with 1.16wmf4 so bug 24313 can be resolved (in lieu of changing $wgHiddenPrefs or such).

#Comment by Nemo bis (talk | contribs)   15:40, 24 August 2010
#Comment by Simetrical (talk | contribs)   17:01, 24 August 2010

(starting at 18:27:29)

#Comment by Gurch (talk | contribs)   22:10, 14 July 2010

Great, now I have to tick a box every time I edit again. Giving serious thought to writing a complete MediaWiki UI replacement for myself...

#Comment by Simetrical (talk | contribs)   22:16, 14 July 2010

If you're editing efficiently, you're probably using Alt-S to save already. Hitting Alt-I first shouldn't be so hard.

#Comment by Gurch (talk | contribs)   11:03, 15 July 2010

Alt-Shift-I on Firefox, you try hitting that with one hand :D

#Comment by Simetrical (talk | contribs)   15:04, 15 July 2010

Easy, with your right hand. If you've just finished filling out the edit summary, you have both hands on the keyboard anyway.

#Comment by Mike.lifeguard (talk | contribs)   15:11, 15 July 2010

You assume the right side of the keyboard has a (working) alt and shift key :D

#Comment by Platonides (talk | contribs)   18:48, 15 July 2010

Which many keyboards don't have.

#Comment by Simetrical (talk | contribs)   18:59, 15 July 2010

So use two hands (= normal typing position), or use a JS one-liner.

#Comment by Platonides (talk | contribs)   23:13, 14 July 2010

Just add to your monobook.js/vector.js

 addOnloadHook(function() { if (wgAction=="edit") document.getElementById("wpMinoredit").checked=true; });
#Comment by Nemo bis (talk | contribs)   18:25, 1 November 2010

Per bug 24313 comment 22 by Tim, someone should revert this and change en.wiki configuration only...

Status & tagging log