Skip to content

Conversation

@malmstein
Copy link
Contributor

@malmstein malmstein commented Jul 17, 2024

Task/Issue URL: https://app.asana.com/0/1157893581871903/1207816154536989/f

Description

Delay the addition of the FocusedView until it’s necessary

Steps to test this PR

Enable FocusedView

  • Fresh install, add some favourites
  • Visit a site so it’s setup for the next time the app opens
  • Force close and open it up again
  • Tap on the omnibar
  • Verify that FocusedView is visible
  • Open New Tab
  • Visit a site
  • Tap on the omnibar
  • Verify FocusedView is visible

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @malmstein and the rest of your teammates on Graphite Graphite

@malmstein malmstein marked this pull request as ready for review July 17, 2024 09:48
@anikiki anikiki self-assigned this Jul 17, 2024
Copy link
Contributor

@anikiki anikiki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@malmstein malmstein merged commit 6d8de9a into develop Jul 17, 2024
@malmstein malmstein deleted the feature/david/07-17-delay_adding_focusedview_until_it_s_necessary branch July 17, 2024 11:58
@malmstein
Copy link
Contributor Author

Thanks @anikiki !

anikiki pushed a commit that referenced this pull request Jul 17, 2024
Task/Issue URL:
https://app.asana.com/0/1157893581871903/1207816154536989/f

### Description
Delay the addition of the FocusedView until it’s necessary

### Steps to test this PR

_Enable FocusedView_
- [x] Fresh install, add some favourites
- [x] Visit a site so it’s setup for the next time the app opens
- [x] Force close and open it up again
- [x] Tap on the omnibar
- [x] Verify that FocusedView is visible
- [x] Open New Tab
- [x] Visit a site
- [x] Tap on the omnibar
- [x] Verify FocusedView is visible
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants