Skip to content
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

Feature FXIOS-7024 [v108] Add feature to reload web page from source, ignoring cache #12231

Merged

Conversation

MincDev
Copy link
Contributor

@MincDev MincDev commented Oct 26, 2022

Resolve #7024 by adding a feature to allow users to reload the page, ignoring the cached site data. This feature adds an extra menu button on the long press of the reload button. The additional button allows a "hard refresh" of the page similar to "Ctrl + F5" on desktop.

@MincDev MincDev changed the title Add feature to reload web page from source, ignoring cache Feature FXIOS-7024 [v108] Add feature to reload web page from source, ignoring cache Oct 26, 2022
Client/Frontend/Browser/Tab Management/Tab.swift Outdated Show resolved Hide resolved
Client/Frontend/Browser/Tab Management/Tab.swift Outdated Show resolved Hide resolved
Client/Frontend/Browser/Tab Management/Tab.swift Outdated Show resolved Hide resolved
Client/Frontend/Strings.swift Outdated Show resolved Hide resolved
@mergify
Copy link
Contributor

mergify bot commented Oct 27, 2022

This pull request has conflicts when rebasing. Could you fix it @MincDev?

@MincDev MincDev requested a review from lmarceau October 27, 2022 13:38
Copy link
Contributor

@lmarceau lmarceau left a comment

Choose a reason for hiding this comment

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

Almost there 💪 few more comments.

Client.xcodeproj/project.pbxproj Outdated Show resolved Hide resolved
Client/Frontend/Browser/Tab Management/Tab.swift Outdated Show resolved Hide resolved
Client/Frontend/Browser/Tab Management/Tab.swift Outdated Show resolved Hide resolved
Client/Frontend/Browser/Tab Management/Tab.swift Outdated Show resolved Hide resolved
Client/Frontend/Browser/Tab Management/Tab.swift Outdated Show resolved Hide resolved
Client/Frontend/Browser/Tab Management/Tab.swift Outdated Show resolved Hide resolved
MincDev and others added 3 commits October 29, 2022 13:20
These suggestions were minor formatting fixes and code improvements by @Imarceau

Co-authored-by: lmarceau <lmarceau@mozilla.com>
@mergify
Copy link
Contributor

mergify bot commented Oct 29, 2022

This pull request has conflicts when rebasing. Could you fix it @MincDev?

@MincDev
Copy link
Contributor Author

MincDev commented Oct 29, 2022

@lmarceau I believe it should be resolved now. I had to recommit indentation fix, because after pulling your suggested changes it was back on tabs, and I accidentally committed like that again. It was probably since you made changes on my original tab indented code? Not sure, but all should be okay now :)

@MincDev MincDev requested a review from lmarceau October 29, 2022 11:54
@lmarceau
Copy link
Contributor

Build is green here

@lmarceau lmarceau merged commit 4d49619 into mozilla-mobile:main Oct 31, 2022
@MincDev MincDev deleted the feature/7024-feature-reload-ignoring-cache branch October 31, 2022 16:07
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.

Add ability to "reload without caching" (long press gesture on reload button)
2 participants