Last modified: 2014-05-22 08:46:26 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 T60311, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 58311 - On page for files that don't exist, edit icon shouldn't open editor for articles
On page for files that don't exist, edit icon shouldn't open editor for articles
Status: RESOLVED FIXED
Product: MobileFrontend
Classification: Unclassified
Feature requests (Other open bugs)
unspecified
All All
: Normal enhancement
: ---
Assigned To: Jared Flores
http://www.google-melange.com/gci/tas...
gci2013
: design
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-12-11 05:55 UTC by Andrew Green
Modified: 2014-05-22 08:46 UTC (History)
15 users (show)

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


Attachments

Description Andrew Green 2013-12-11 05:55:26 UTC
If you go to the page of a file that doesn't exist, you get this message: "No file by this name exists, but you can upload it." The message includes a nice link to Special:Upload. However, on the same page, the pencil icon opens the editor. Instead, it should probably take you to Special:Upload instead.
Comment 1 Bingle 2013-12-11 06:05:28 UTC
Prioritization and scheduling of this bug is tracked on Mingle card https://wikimedia.mingle.thoughtworks.com/projects/mobile/cards/1497
Comment 2 Jon 2013-12-24 17:07:58 UTC
Kenan / designers what do you think?
Comment 3 Jared Zimmerman (WMF) 2013-12-24 20:48:21 UTC
I think we should have an upload icon for this case rather than an edit icon.
Comment 4 Jon 2013-12-24 20:52:49 UTC
Jared so how about this as a first step:
Visiting https://en.m.wikipedia.org/wiki/File:Foozajashsaash.png (a file that doesn't exist)
shows a page actions bar without an edit button. Clicking upload starts the normal upload workflow but saves it to the same file name as given in the URL?

Quim this could be a good task for Google Code in.
Comment 5 Jon 2013-12-24 20:53:12 UTC
(upload being the existing upload icon in the page action bar)
Comment 6 Jared Zimmerman (WMF) 2013-12-24 21:11:30 UTC
That sounds fine, we could suppress the normal edit icon in this case? also we'd switch the image icon not to be the locked state, correct?
Comment 7 Quim Gil 2013-12-24 22:26:56 UTC
Created a GCI task. Please leave the task as clear as possible to the eventual CI student. Thank you!
Comment 8 Jared Flores 2013-12-26 19:46:07 UTC
Just to be clear, is this what you want?
-Disable the edit button and switch the icon to the disabled edit icon
-Unlock the upload button, and trigger the normal upload workflow
Comment 9 Quim Gil 2013-12-27 06:07:42 UTC
Yes JaredF, this is what I also understand. 

fyi, Jared Flores is a Google Code-in student that has started working on this task.
Comment 10 Jared Flores 2013-12-27 23:07:36 UTC
Hi,
Where exactly would I find the handling for non-existent files in MobileFrontend? I've been told it might be in \includes\skins\SkinMobileBeta.php, but I've been at a loss of where to find it.
Thanks in Advanced,
Jared Flores
Comment 11 Jared Flores 2013-12-28 04:32:23 UTC
Nevermind, I believe I've found it. Patch will be uploaded once my testing environment starts working again
Comment 12 Jared Flores 2013-12-28 09:50:06 UTC
Hi,
I've managed to disable the edit button already, but what exactly would you like me to do with the upload button?
I've already unlocked it, but I'm not sure where to go from there. Do you want me to link the button with Special:Upload? Or would you like me to modify the PhotoUpload.js to include a function for non-existent files. The latter is a tad bit harder, but if that's what you would like, I'll do it
Thanks in Advanced,
Jared Flores
Comment 13 Gerrit Notification Bot 2013-12-29 02:11:22 UTC
Change 104324 had a related patch set uploaded by Jcf2000:
On "File does not exist" disable edit, enable upload

https://gerrit.wikimedia.org/r/104324
Comment 14 Gerrit Notification Bot 2014-03-04 23:01:38 UTC
Change 104324 abandoned by Jdlrobson:
On "File does not exist" disable edit, enable upload

Reason:
At request of submitter. Feel free to re-open this or use as a basis for a future fix to this problem! :)

https://gerrit.wikimedia.org/r/104324
Comment 15 Gerrit Notification Bot 2014-05-21 11:11:43 UTC
Change 134589 had a related patch set uploaded by Florianschmidtwelzow:
Implement upload via "Add image" on New file pages

https://gerrit.wikimedia.org/r/134589
Comment 16 Gerrit Notification Bot 2014-05-22 08:18:03 UTC
Change 134589 merged by jenkins-bot:
Implement upload via "Add image" on New file pages

https://gerrit.wikimedia.org/r/134589

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


Navigation
Links