make "does not exist" warning in file viewer friendlier

VERIFIED FIXED in 6.0.11

Status

addons.mozilla.org Graveyard
Admin/Editor Tools
P5
enhancement
VERIFIED FIXED
7 years ago
2 years ago

People

(Reporter: clouserw, Assigned: andym)

Tracking

unspecified
6.0.11

Details

Attachments

(1 attachment)

(Reporter)

Description

7 years ago
Right now the warning says something like "content/findinnumbers.css does not exist in file 48935."  I'm suggesting we make that "$file does not exist in the older version" or "...the newer version".  Even if that is just comparing which number is bigger.  What do you think?
(Assignee)

Comment 1

7 years ago
I like that. Thinking I really should just show a whole pile of + or - instead though.
(Reporter)

Comment 2

7 years ago
(In reply to comment #1)
> I like that. Thinking I really should just show a whole pile of + or -
> instead though.

I like that even better, too!

Comment 3

7 years ago
Also, we need to think of a way to show deleted files in the file differ.
(Reporter)

Updated

7 years ago
Target Milestone: Q2 2011 → 6.0.11
(Assignee)

Comment 4

7 years ago
Deleted files are show in a second tree (if present) below the first tree. 

https://github.com/jbalogh/zamboni/commit/b7a38434034fceb7d20e5103b9ecdb9b5c5b1925
https://github.com/jbalogh/zamboni/commit/d60b5dce7a80c57430aff3b00083eb3b4d658e35
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
(Assignee)

Comment 5

7 years ago
Showing the different search engine file as a deleted file.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 6

7 years ago
https://github.com/jbalogh/zamboni/commit/7b5ac5e
Status: REOPENED → RESOLVED
Last Resolved: 7 years ago7 years ago
Resolution: --- → FIXED

Comment 7

7 years ago
verified fixed @ https://addons.allizom.org/en-US/firefox/files/compare/119615...79034/file/install.rdf#top
Status: RESOLVED → VERIFIED

Comment 8

7 years ago
Created attachment 535166 [details]
post-fix screenshot
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.