X
Tap here to go to the mobile version of the site.

Support Forum

A bug has landed on the page info window > media > sort by size… is no longer sorting by size.

Posted

the page info window > media > sort by size… is no longer sorting by size. Clicking the size header does seem to change the order, but no longer by size. It worked fine before the last update a few days ago.

the page info window > media > sort by size… is no longer sorting by size. Clicking the size header does seem to change the order, but no longer by size. It worked fine before the last update a few days ago.
Quote

Additional System Details

Installed Plug-ins

  • Shockwave Flash 19.0 r0

Application

  • User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:67.0) Gecko/20100101 Firefox/67.0

More Information

cor-el
  • Top 10 Contributor
  • Moderator
17347 solutions 156811 answers

Helpful Reply

I can confirm this and it has already been reported as a bug (regression).

  • Bug 1554399 - incorrect media sort by size in view page info

(please do not comment in bug reports
https://bugzilla.mozilla.org/page.cgi?id=etiquette.html
)

I can confirm this and it has already been reported as a bug (regression). *[https://bugzilla.mozilla.org/show_bug.cgi?id=1554399 Bug 1554399] - incorrect media sort by size in view page info (<i>please do not comment in bug reports<br>https://bugzilla.mozilla.org/page.cgi?id=etiquette.html</i>)
Was this helpful to you? 1
Quote
jscher2000
  • Top 10 Contributor
8583 solutions 70209 answers

It seems to re-use the last sort order.

But thanks for mentioning the problem, as I didn't realize I could add this column. :-)

It seems to re-use the last sort order. But thanks for mentioning the problem, as I didn't realize I could add this column. :-)
Was this helpful to you?
Quote
cor-el
  • Top 10 Contributor
  • Moderator
17347 solutions 156811 answers

It is not the last sort order, but things got messed up after converting older code to fluent. It probably tries to compare bytes stored in this field (numbers instead of the string representation). Sorting for numbers used to be done via number compare (a-b) and not via text compare

Compare how it is done in Firefox 60 ESR gImageView.onPageMediaSort https://dxr.mozilla.org/mozilla-esr60/source/browser/base/content/pageinfo/pageInfo.js#178

It is not the last sort order, but things got messed up after converting older code to fluent. It probably tries to compare bytes stored in this field (numbers instead of the string representation). Sorting for numbers used to be done via number compare (a-b) and not via text compare *https://dxr.mozilla.org/mozilla-release/source/browser/base/content/pageinfo/pageInfo.js#189 Compare how it is done in Firefox 60 ESR gImageView.onPageMediaSort https://dxr.mozilla.org/mozilla-esr60/source/browser/base/content/pageinfo/pageInfo.js#178
Was this helpful to you?
Quote
Ask a question

You must log in to your account to reply to posts. Please start a new question, if you do not have an account yet.