Opened 13 years ago

Closed 13 years ago

#1172 closed Bug (fixed)

web ui's inspector is inconsistent with unavailable text

Reported by: livings124 Owned by: Paweł
Priority: Normal Milestone: 1.40
Component: Web Client Version: 1.32
Severity: Normal Keywords:
Cc:

Description

  1. In the web ui's inspector, some fields show N/A when it's not available, others are just empty string. Additionally, this should be consistent between when none is selected and a selected transfer doesn't have that info.
  1. Values should be N/A, not 0, when no transfer is selected.
  1. N/A should be a placeholder (gray text).

Attachments (1)

n-as_fixed.diff (1.1 KB) - added by etsuffer 13 years ago.
diff against 6482 revision

Download all attachments as: .zip

Change History (10)

comment:1 Changed 13 years ago by etsuffer

N/A values already fixed in #6475 http://trac.transmissionbt.com/changeset/6475 So if you select single torrent you will see its comment, creator, etc. (was always N/a before) Selecting multiple torrents puts N\a's there

comment:2 Changed 13 years ago by livings124

With that commit if there are no comment or creator, it shows the empty string instead of N/A. All 3 bullet points are still valid.

comment:3 Changed 13 years ago by etsuffer

'N/A' sign looks to me like "couldn't be displayed". But if you think its right I could fix that

Also I tried selecting/deseleting torrents but couldn't repeat the bug with 0's in torrent inspector (described in 2nd bullet point)

Changed 13 years ago by etsuffer

diff against 6482 revision

comment:4 Changed 13 years ago by livings124

For 2, I mean under the network tab of the inspector: downloaded amounts are 0 bytes, for example.

comment:5 Changed 13 years ago by charles

r6656 fixes the torrent.js typo that's pointed out in n-as_fixed.diff

comment:6 Changed 13 years ago by charles

  • Milestone changed from None Set to 1.40

I think (2) is fixed in r6662

livings124: is there anything else left to do for this ticket, or can it be closed?

comment:7 Changed 13 years ago by livings124

  1. N/A should be a placeholder (gray text).

comment:8 Changed 13 years ago by charles

  • Owner changed from Gimp to Paweł
  1. is fixed in r6790

comment:9 Changed 13 years ago by charles

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.