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] Refresh Feature for a single view #1077

Merged
merged 2 commits into from Sep 15, 2023

Conversation

ParamBirje
Copy link
Contributor

✨ Pull Request

📓 Referenced Issue

Fixes #1064

ℹ️ About the PR

As described in the issue, there was a need of refreshing only a specific view without the need of using the console. The current state of the app contains a global refresh button on the left of the URL bar, which triggers a reload for every view in the app.

To fix the issue, a new button is added in the view specific toolbar, that will refresh that view without hampering other views.

🖼️ Testing Scenarios / Screenshots

Test: Changed some element's text in different views (using inspect element) and tried refreshing the views one at a time.

refresh-button-closeup

refresh-view

@CLAassistant
Copy link

CLAassistant commented Sep 14, 2023

CLA assistant check
All committers have signed the CLA.

Copy link
Collaborator

@manojVivek manojVivek left a comment

Choose a reason for hiding this comment

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

Fantastic work on this! Looking forward to more like this. 🚀

@manojVivek manojVivek merged commit 468ade9 into responsively-org:main Sep 15, 2023
4 checks passed
@manojVivek
Copy link
Collaborator

@all-contributors Please add @ParamBirje for code.

@allcontributors
Copy link
Contributor

@manojVivek

I've put up a pull request to add @ParamBirje! 🎉

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.

Refresh a single view
3 participants