Show number of attachments next to attachment heading on wiki page

[Ryan J Ollos <ryano@…>]

As originally discussed in comment:35:ticket:8295:

Since the attachments on a wiki page are hidden in a collapsed section when the page is initially loaded, it would be useful to show the number of attachments next to the heading, e.g. Attachments (6). Without this, there is no immediate way to see that the page has attachments.

This may also be useful to show on the Ticket page, though it seems less important in that case since the attachments section is initially expanded.

[Ryan J Ollos <ryano@…>]

Replying to Ryan J Ollos <ryano@…>:

Without this, there is no immediate way to see that the page has attachments.

I should have said, without this, there is no immediate way to see the number of attachments. Alternatively, we could just show the number of attachments when the foldable section is collapsed.

I'm on the fence as to whether this is a good idea, or all that useful. I thought I'd give the implementation a try anyway. See attached patch.

---

---

[lkraav <leho@…>]

Probably a very good point. I today also noticed with my last comment:67:ticket:10012 upload that if i have "Comments only" checked, then I will have no indication of attachments being sent at all. Felt confusing. I'm pretty sure this is another thing related to the "we need a sidebar" proposal #10739. I guess a position:fixed header control panel of some sort is also an option, but sidebars are definitely more versatile.

[Christian Boos]

Replying to Ryan J Ollos <ryano@…>:

This may also be useful to show on the Ticket page, though it seems less important in that case since the attachments section is initially expanded.

… but this would enable us to have that section collapsed by default as well.

[Ryan J Ollos <ryano@…>]

Replying to cboos:

… but this would enable us to have that section collapsed by default as well.

+1 from me, but there had previously been some reservation about making that change: comment:33:ticket:8295. Maybe you are less fearful of revolts? ;)

I also noted in comment:32:ticket:8295, that if we were to make the ticket attachments section folded by default, we might want to bring the attachment button outside of the foldable area. Thoughts?

[Ryan J Ollos <ryano@…>]

A couple of thoughts:

• After staring at this for a while, it starts to look out of place that the Change History doesn't indicate the number of entries as well. Since that section is initially folded, maybe that is a feature we'd want as well?
• Should the attachments quickjump at the bottom of the wiki page also cause a folded Attachments section to be unfolded?

[Ryan J Ollos <ryano@…>]

Trivial change from previous patch - Ticket's attachments section is now initially folded.

[Christian Boos]

• After staring at this for a while, it starts to look out of place that the Change History doesn't indicate the number of entries as well. Since that section is initially folded, maybe that is a feature we'd want as well?

Yes, that was my reasoning as well.

• Should the attachments quickjump at the bottom of the wiki page also cause a folded Attachments section to be unfolded?

Could be, yes. Note that I'm removing the "Modify" quick jump, but the one for the attachments (at the bottom of the page) doesn't get so in the way.

I'm toying with the idea of having a ticket view going like this:

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      |                                                       |
      |                                                       |
      |                                                       |
      '-------------------------------------------------------'

      > Attachments (2) [+]          [ Edit ] [x] Side-by-side

      > Change History

      Add a comment
      .-------------------------------------------------------.
      |                                                       |
      |                                                       |
      '-------------------------------------------------------'

      [ Preview ]   [ Edit ](*)                    [ Submit ]

(*) Maybe with a reprise of the edit button at the bottom.

(but I'm afraid I won't have the time to experiment with this until I'm back from holidays, 2nd week of July)

[Christian Boos]

I have a few follow-ups on this (which in turn will be polished as part of #10012).

[Christian Boos]

See repos:cboos.git:ticket10743/t10743-r11098-1.patch-apply

[Christian Boos]

Committed the above changesets in r11126, r11127 and r11128.

The last needed a bit of additional clean-up. I removed the whole onfocus stuff, I don't think it ever worked (at least it doesn't in 0.12.4dev either, e.g. here on Babel or demo-0.12).