Closed
Bug 737348
Opened 12 years ago
Closed 12 years ago
Filelink notifification should display a 'What's this ?' link to a Filelink feature description
Categories
(Thunderbird :: Message Compose Window, defect)
Tracking
(thunderbird13 fixed)
RESOLVED
FIXED
Thunderbird 14.0
Tracking | Status | |
---|---|---|
thunderbird13 | --- | fixed |
People
(Reporter: jb, Assigned: mconley)
References
Details
Attachments
(3 files, 3 obsolete files)
75.85 KB,
image/png
|
Details | |
4.85 KB,
patch
|
Details | Diff | Splinter Review | |
3.07 KB,
patch
|
standard8
:
approval-comm-aurora+
|
Details | Diff | Splinter Review |
The current Filelink notification bar can appear impromptu without the user ever heard of Filelink. In this context, a link should point to the feature description, shown in a popup window. The help content is TBD but needs to be localized.
Reporter | ||
Updated•12 years ago
|
Assignee: nobody → bwinton
Assignee | ||
Comment 1•12 years ago
|
||
Blake and I discussed this change before string freeze, and I believe the solution we came up with was to use a question mark icon within a button. See attached screenshot.
Reporter | ||
Comment 2•12 years ago
|
||
Yes, that should be just fine. Jenzed, can you come up with some feature documentation to populate the popup window ?
Updated•12 years ago
|
Assignee: bwinton → mconley
Assignee | ||
Comment 3•12 years ago
|
||
Andreas: Do we have a stick question mark, or "help" icon that we can use here? -Mike
Assignee | ||
Comment 4•12 years ago
|
||
This patch is specifically for comm-aurora, because it hard-codes in a "?" string for the new button I've added (this is the compromise that bwinton and I came up with, since we obviously don't want to add any new strings). For comm-central, I will prepare a separate patch that will include a "Learn more" string instead of the hard-coded question mark. When clicking the "?", notice that the URL loads in a contentTab. Currently, I'm pointing it at https://support.mozillamessaging.com/, but clearly we need to find a real URL for it. As soon as someone lets me know what the real URL will be, I'll update the patch.
Assignee | ||
Comment 5•12 years ago
|
||
So, who is going to create the Filelink "Learn more" page, and what URL will it be at?
Reporter | ||
Comment 6•12 years ago
|
||
I can provide a text. As for the URL, I have no idea: can Sancus or Mark propose something ?
Assignee | ||
Comment 7•12 years ago
|
||
Bump
Comment 8•12 years ago
|
||
I'm sure we can find somewhere on our vast web server to put a page, given content.
Comment 9•12 years ago
|
||
Though, on second thought, isn't this really a good use of a knowledge base article?
Assignee | ||
Comment 10•12 years ago
|
||
(In reply to Andrei Hajdukewycz [:sancus] from comment #9) > Though, on second thought, isn't this really a good use of a knowledge base > article? Yeah, actually, I think you're totally right. Roland / Jen: what would be the best way for Jb to set up a "Filelink" page on support.mozillamessaging.com? -Mike
Comment 11•12 years ago
|
||
In general, you can create a new KB page through this link: https://support.mozillamessaging.com/en-US/kb/new (after creating an account). However I'm glad to do it for you. Stub page (use this URL as the link from the icon): http://support.mozillamessaging.com/kb/file-link Jb: you can edit that page, or send me the content you'd like to see, or just wait until I start doing the docs for the related release.
Assignee | ||
Comment 12•12 years ago
|
||
Sorry Blake, more for the review queue. I'll try to steal a few more from you today. :)
Attachment #608816 -
Attachment is obsolete: true
Attachment #610125 -
Flags: ui-review?(bwinton)
Attachment #610125 -
Flags: review?(bwinton)
Assignee | ||
Comment 13•12 years ago
|
||
Comment on attachment 610125 [details] [diff] [review] Patch for comm-aurora I may have found a "Learn More..." string we can re-use in comm-aurora. Once I get the OK to use it, I'll pump out a new patch.
Attachment #610125 -
Flags: ui-review?(bwinton)
Attachment #610125 -
Flags: review?(bwinton)
Assignee | ||
Comment 14•12 years ago
|
||
This patch introduces a new "Learn More..." string for the compose window.
Attachment #610131 -
Flags: ui-review?(bwinton)
Attachment #610131 -
Flags: review?(bwinton)
Reporter | ||
Comment 15•12 years ago
|
||
Suggested text is here: https://support.mozillamessaging.com/en-US/kb/file-link/discuss/516 jenzed: can you please proof and make sure this is understandable ? And of course, comment !
Comment 16•12 years ago
|
||
Comment on attachment 610131 [details] [diff] [review] Patch for comm-central The UI seems fine, so ui-r=me. >+++ b/mail/locales/en-US/chrome/messenger/messengercompose/composeMsgs.properties >@@ -457,8 +457,10 @@ cloudAttachmentListFooter=%1$S makes it > > ## LOCALIZATION NOTE(cloudAttachmentListItem): A line of text describing a cloud > ## attachment, to be inserted into the message body. Do not translate the words > ## %1$S, %2$S, %3$S, or %4$S. %1$S is the attachment name, %2$S is its size, > ## %3$S is the name of the cloud storage service, and %4$S is the link to the > ## attachment. > cloudAttachmentListItem=* %1$S (%2$S) hosted on %3$S: %4$S > >+learnMore.label=Learn More⦠>+learnMore.accesskey=L So, I think these lines should go up before the bigFileShare.label lines. And, it looks like "bigFileShare.accesskey=l", so I worry about using "L" for Learn More… How about "m" instead? r=me with those changes. Later, Blake.
Attachment #610131 -
Flags: ui-review?(bwinton)
Attachment #610131 -
Flags: ui-review+
Attachment #610131 -
Flags: review?(bwinton)
Attachment #610131 -
Flags: review+
Assignee | ||
Comment 17•12 years ago
|
||
Thanks! Fixed those issues.
Attachment #610131 -
Attachment is obsolete: true
Assignee | ||
Updated•12 years ago
|
Attachment #610879 -
Attachment description: Patch v2 (carrying over r+/ui-r+ from bwinton) → Patch v2 for comm-central (carrying over r+/ui-r+ from bwinton)
Assignee | ||
Comment 18•12 years ago
|
||
This patch is nearly identical to the patch for comm-central, but has two important caveats: 1) We borrow a string from messenger.properties 2) We don't have an access key to the "Learn More..." button. But of those issues are dealt with in the comm-central patch.
Attachment #610125 -
Attachment is obsolete: true
Attachment #610886 -
Flags: ui-review?(bwinton)
Attachment #610886 -
Flags: review?(bwinton)
Assignee | ||
Comment 19•12 years ago
|
||
Landed in comm-central as http://hg.mozilla.org/comm-central/rev/ccb4559ce25b
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 14.0
Assignee | ||
Comment 20•12 years ago
|
||
Comment on attachment 610886 [details] [diff] [review] Patch v2 for comm-aurora (r+ / ui-r+ from bwinton over irc) bwinton gave me the ol' thumbs up over IRC.
Attachment #610886 -
Attachment description: Patch v2 for comm-aurora → Patch v2 for comm-aurora (r+ / ui-r+ from bwinton over irc)
Attachment #610886 -
Flags: ui-review?(bwinton)
Attachment #610886 -
Flags: review?(bwinton)
Attachment #610886 -
Flags: approval-comm-aurora?
Reporter | ||
Comment 21•12 years ago
|
||
The feature name is Filelink and should be referred as Thunderbird Filelink in external communications. The support article title should therefore be 'Thunderbird Filelink' and not 'File Link' (2 words) https://support.mozillamessaging.com/en-US/kb/file-link
Updated•12 years ago
|
Attachment #610886 -
Flags: approval-comm-aurora? → approval-comm-aurora+
Assignee | ||
Comment 22•12 years ago
|
||
Committed to comm-aurora as http://hg.mozilla.org/releases/comm-aurora/rev/a183704a5449
status-thunderbird13:
--- → fixed
Assignee | ||
Comment 23•12 years ago
|
||
(In reply to Jb Piacentino from comment #21) > The feature name is Filelink and should be referred as Thunderbird Filelink > in external communications. > The support article title should therefore be 'Thunderbird Filelink' and not > 'File Link' (2 words) > https://support.mozillamessaging.com/en-US/kb/file-link Jen: Are you able to update the support page to reflect this? -Mike
Comment 24•12 years ago
|
||
The article title and contents are fixed.
Reporter | ||
Comment 25•12 years ago
|
||
Two more comments on this: - I think it would be nicer if we could have a 'popup' window, instead of a tab: the user can read the information, close the window, while still having the compose window visible in the background - Is the SUMO article the best place to display this help text ? This page is full of navigation items that can be confusing to users. Can we locate this somewhere else and format is in a way that better correspond to a help text? Finally, the feature name is Thunderbird Filelink. The current page should be renamed or a new one created with the right title (and the old one deleted).
Assignee | ||
Comment 26•12 years ago
|
||
(In reply to Jb Piacentino from comment #25) > - Is the SUMO article the best place to display this help text ? This page > is full of navigation items that can be confusing to users. Can we locate > this somewhere else and format is in a way that better correspond to a help > text? There are other instances where we directly link to the knowledge base from within Thunderbird. For example, when a plug-in crashes in a content tab, a support link appears in the "broken plugin" content frame. That support link is a knowledge base article that opens up in a new content tab. Another case is when the user goes to Help > Help Contents - we open up the knowledge base in the default browser. I would argue that having the page be hosted on the knowledge base makes sense for two reasons: 1) It's internally consistent with the other two cases I listed above 2) It might allow for an easier on-ramp for contribution, since (I believe) we already have a fairly active community working on the knowledge base. I'm, however, always open to argument. :) -Mike
Comment 27•12 years ago
|
||
I think adopting another platform / mechanism for deploying documentation is a big project and can't be approached on a one-off basis. If we wanted to do this, we would have to scrape content from the knowledge base so that we don't have to manually update two versions. Regarding "Thunderbird Filelink" - sorry, I added an erroneous capital - I'll fix. Do you really mean you want all instances where we refer to "Filelink" to be changed to "Thunderbird Filelink"? As in, "Thunderbird Filelink enables you to .... With Thunderbird Filelink ... To access Thunderbird Filelink..."
Reporter | ||
Comment 28•12 years ago
|
||
(In reply to jenzed from comment #27) > I think adopting another platform / mechanism for deploying documentation is > a big project and can't be approached on a one-off basis. If we wanted to do > this, we would have to scrape content from the knowledge base so that we > don't have to manually update two versions. Fair enough. > > Regarding "Thunderbird Filelink" - sorry, I added an erroneous capital - > I'll fix. Do you really mean you want all instances where we refer to > "Filelink" to be changed to "Thunderbird Filelink"? As in, "Thunderbird > Filelink enables you to .... With Thunderbird Filelink ... To access > Thunderbird Filelink..." Yes. "Thunderbird Filelink" is the feature name. It might be shortened to Filelink if the context is not ambiguous or has previously been defined. No other spelling should be used, IMO.
Reporter | ||
Comment 29•12 years ago
|
||
(In reply to Mike Conley (:mconley) from comment #26) What about the popup iso a tab ?
Assignee | ||
Comment 30•12 years ago
|
||
(In reply to Jb Piacentino from comment #29) > (In reply to Mike Conley (:mconley) from comment #26) > What about the popup iso a tab ? Blake - thoughts? Should I file a bug to open the link in the browser instead?
Comment 31•12 years ago
|
||
A popup that came out of the "Learn more" button? I'm not a fan of that, and we don't do that sort of UI anywhere else as far as I know… The Account Provisioner stuff all opens in the browser, but that's mostly because it's a modal dialog, and you wouldn't be able to read the page if we tried to open it in a tab. Further, I don't think hosting a full, editable, SuMo page in a popup would work out well, since it could change length at any time. So I think the current solution is the one we want to go with, even though it will work out better once we get compose-in-a-tab. Perhaps focusing the newly-opened content tab would help address JB's concerns? Thanks, Blake.
Assignee | ||
Comment 32•12 years ago
|
||
(In reply to Blake Winton (:bwinton - Thunderbird UX) from comment #31) > So I think the current solution is the one we want to go with, even though > it will work out better once we get compose-in-a-tab. Perhaps focusing the > newly-opened content tab would help address JB's concerns? Just a note that we already focus the new content tab after spawning it.
You need to log in
before you can comment on or make changes to this bug.
Description
•