-
Notifications
You must be signed in to change notification settings - Fork 60
feat: history view #130
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
feat: history view #130
Conversation
|
@croossin is the goal to implement all the browser settings pages in regular views but as swift UI rather than webpages or is it going to be a mix? |
My POV is that all views should be written in SwiftUI |
|
This is really good @croossin. Can you explain the need |
@kenenisa The
|
|
I was trying to avoid the history record getting too big with repeated entries but i guess there is no other way to do it so this should be fine. |
|
@yonaries will clean up the ui |
|
@kenenisa @yonaries -- landed on just leveraging the existing |
Add History View Feature
Overview
Implements a history view that preserves the actual sequence of visits, in conjunction with the previous consolidated-only approach.
Key Changes
New Features
Implementation
HistoryVisitfor individual chronological entriesora://, etc.) from historyUsage
Cmd+Yto open history tabDemo
CleanShot.2025-09-28.at.17.02.42.mp4
Help needed
Align on UX for bulk deletion, selection, appropriate debouncing (i.e. frequency of how/when visits are stored)