Skip to content
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion .swift-version

This file was deleted.

4 changes: 2 additions & 2 deletions special-pages/pages/history/app/components/Item.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ export const Item = memo(
* @param {string} props.url - The text to be displayed for the item.
* @param {string} props.domain - The text to be displayed for the domain
* @param {number} props.kind - The kind or type of the item that determines its visual style.
* @param {string} props.dateTimeOfDay - the time of day, like 11.00am.
* @param {string} [props.dateTimeOfDay] - the time of day, like 11.00am.
* @param {string} props.dateRelativeDay - the time of day, like 11.00am.
* @param {string|null} props.etldPlusOne
* @param {number} props.index - original index
Expand Down Expand Up @@ -63,7 +63,7 @@ export const Item = memo(
<span class={styles.domain} data-testid="Item.domain" title={props.domain}>
{props.domain}
</span>
<span class={styles.time}>{dateTimeOfDay}</span>
{dateTimeOfDay && <span className={styles.time}>{dateTimeOfDay}</span>}
<button class={styles.dots} data-action={BTN_ACTION_ENTRIES_MENU} data-index={index} value={props.id} tabindex={-1}>
<Dots />
</button>
Expand Down
3 changes: 3 additions & 0 deletions special-pages/pages/history/app/components/Sidebar.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ const iconMap = {
saturday: 'icons/day.svg',
sunday: 'icons/day.svg',
older: 'icons/older.svg',
sites: 'icons/sites.svg',
};

/** @type {Record<RangeId, (t: (s: keyof json) => string) => string>} */
Expand All @@ -43,6 +44,7 @@ const titleMap = {
saturday: (t) => t('range_saturday'),
sunday: (t) => t('range_sunday'),
older: (t) => t('range_older'),
sites: (t) => t('range_sites'),
};

/**
Expand Down Expand Up @@ -182,6 +184,7 @@ function labels(range, t) {
case 'thursday':
case 'friday':
case 'saturday':
case 'sites':
case 'sunday':
return { linkLabel: t('show_history_for', { range }), buttonLabel: t('delete_history_for', { range }) };
case 'older':
Expand Down
1 change: 1 addition & 0 deletions special-pages/pages/history/app/history.service.js
Original file line number Diff line number Diff line change
Expand Up @@ -421,6 +421,7 @@ export function toRange(input) {
'sunday',
'recentlyOpened',
'older',
'sites',
];
return valid.includes(input) ? /** @type {import('../types/history.js').RangeId} */ (input) : null;
}
Expand Down
3 changes: 3 additions & 0 deletions special-pages/pages/history/app/mocks/mock-transport.js
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,7 @@ export function mockTransport() {
{ id: 'saturday', count: 1 },
{ id: 'friday', count: 1 },
{ id: 'older', count: 1 },
{ id: 'sites', count: 1 },
],
};

Expand Down Expand Up @@ -268,8 +269,10 @@ function queryResponseFrom(memory, msg) {
const response = asResponse(memory.slice(0, 10), msg.params.offset, msg.params.limit);
const range = msg.params.query.range;
response.value = response.value.map((item) => {
if (!('range' in msg.params.query)) return item; // unreachable
return {
...item,
dateTimeOfDay: msg.params.query.range === 'sites' ? undefined : item.dateTimeOfDay,
title: 'range:' + range + ' ' + item.title,
};
});
Expand Down
4 changes: 4 additions & 0 deletions special-pages/pages/history/app/strings.json
Original file line number Diff line number Diff line change
Expand Up @@ -106,5 +106,9 @@
"range_older": {
"title": "Older",
"note": "Label on a button that shows older history entries."
},
"range_sites": {
"title": "Sites",
"note": "Label on a button that shows which sites have been visited"
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@
"id",
"dateRelativeDay",
"dateShort",
"dateTimeOfDay",
"domain",
"time",
"title",
Expand Down
3 changes: 2 additions & 1 deletion special-pages/pages/history/messages/types/range.json
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,8 @@
"friday",
"saturday",
"sunday",
"older"
"older",
"sites"
]
}
},
Expand Down
19 changes: 19 additions & 0 deletions special-pages/pages/history/public/icons/sites.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
4 changes: 4 additions & 0 deletions special-pages/pages/history/public/locales/en/history.json
Original file line number Diff line number Diff line change
Expand Up @@ -116,5 +116,9 @@
"range_older": {
"title": "Older",
"note": "Label on a button that shows older history entries."
},
"range_sites": {
"title": "Sites",
"note": "Label on a button that shows which sites have been visited"
}
}
5 changes: 3 additions & 2 deletions special-pages/pages/history/types/history.ts
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,8 @@ export type RangeId =
| "friday"
| "saturday"
| "sunday"
| "older";
| "older"
| "sites";
export type QueryKind = SearchTerm | DomainFilter | RangeFilter;
/**
* Indicates the query was triggered before the UI was rendered
Expand Down Expand Up @@ -266,7 +267,7 @@ export interface HistoryItem {
/**
* The time of day in 24-hour format (e.g., '11:01').
*/
dateTimeOfDay: string;
dateTimeOfDay?: string;
/**
* The eTLD+1 version of the domain, representing the domain and its top-level domain (e.g., 'example.com', 'localhost'). This differs from 'domain', which may include subdomains (e.g., 'www.youtube.com').
*/
Expand Down
Loading