History search page should expose the full timestamp |
|||
Issue descriptionThe formatting isn't consistent between the regular and searched history pages. In the regular view, the pages are split by date, with a header separating each day. The time is shown in gray text to the left of each page name. When you search for a term, the headers disappear and the dates are moved to where the times used to be. Hovering over the dates doesn't reveal the time either; it's just gone. The only way to find the time is to go back to your full history and manually comb through it until you find the page. This seems like it should be an easy fix, at least to expose it as hover text if nothing else. Rachel, could you triage this?
,
Sep 23 2016
No, we still show times on the regular page and dates on the search page. On the search page, we use the header (normally used for the date) to show the number of search results. I think the logic is that when you are searching for something, the date is more important than the time since there are far fewer matches to deal with. @bettes, what do you think?
,
Oct 31 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/55469f608ab2c7333fa55e96969e0117505c2141 commit 55469f608ab2c7333fa55e96969e0117505c2141 Author: calamity <calamity@chromium.org> Date: Mon Oct 31 06:56:26 2016 [MD History] Add full time hover text to dates in history items. BUG= 642575 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2454303002 Cr-Commit-Position: refs/heads/master@{#428662} [modify] https://crrev.com/55469f608ab2c7333fa55e96969e0117505c2141/chrome/browser/resources/md_history/app.crisper.js [modify] https://crrev.com/55469f608ab2c7333fa55e96969e0117505c2141/chrome/browser/resources/md_history/app.vulcanized.html [modify] https://crrev.com/55469f608ab2c7333fa55e96969e0117505c2141/chrome/browser/resources/md_history/history_item.html [modify] https://crrev.com/55469f608ab2c7333fa55e96969e0117505c2141/chrome/browser/resources/md_history/history_item.js
,
Nov 2 2016
|
|||
►
Sign in to add a comment |
|||
Comment 1 by rpop@chromium.org
, Sep 21 2016