brave / brave-ios

Brave iOS Browser
https://brave.com
Mozilla Public License 2.0
1.7k stars 441 forks source link

[Follow up to #8701] `NTP SI` is shown on 2nd attempt after browser launch, and then on every 4th instead of every 3rd attempt #8740

Open hffvld opened 7 months ago

hffvld commented 7 months ago

Description:

Follow-up to #8701

Steps to Reproduce

  1. Install and launch Brave
  2. Wait until landed on NTP
  3. Open new NTP (2nd one in total) > Confirm that SI is opened
  4. Open 4 more NTP > Observe that the next SI is shown on every 4th NTP

Actual result:

NTP SI is shown on the 2nd attempt after browser launch, and then on every 4th instead of every 3rd attempt.

1 2
1 2

https://github.com/brave/brave-ios/assets/128532965/db302e73-5da2-4191-8925-9659d0e503eb

Expected result:

As per https://github.com/brave/brave-ios/pull/8702#issue-2100748362:

Reproduces how often: [Easily reproduced, Intermittent Issue]

Easily reproduced

Brave Version:

Brave build: 1.62 (24.2.6.11)
BraveCore: 1.62.158 (121.0.6167.139)

Device details:

iPhone 14 / iOS 17.4 Beta

Website problems only:

Additional Information

iccub commented 7 months ago

@iambrianfung Hi is this behavior correct or a bug?

iambrianfung commented 7 months ago

i think @tmancey has some context. this should be correct behavior default behavior for ios, but possibly soon to change.