Skip to content

Fixes 1225224 - Prompt on clearing History when signed in to Sync#1369

Merged
st3fan merged 1 commit intomasterfrom
st3fan/Bug1225224PromptOnClearSyncedHistory
Dec 17, 2015
Merged

Fixes 1225224 - Prompt on clearing History when signed in to Sync#1369
st3fan merged 1 commit intomasterfrom
st3fan/Bug1225224PromptOnClearSyncedHistory

Conversation

@st3fan
Copy link
Contributor

@st3fan st3fan commented Dec 17, 2015

This patch shows a confirmation dialog when you Clear Private Data with History selected when logged in to Sync. It will show the following:

screenshot 2015-12-17 15 28 09

Copy link
Contributor

Choose a reason for hiding this comment

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

I'd call this clearSyncedHistory; it won't propagate to the server yet.

@rnewman
Copy link
Contributor

rnewman commented Dec 17, 2015

Looks good with those changes, current strings from d5b0d46, and a follow-up for more precise conditional.

@st3fan st3fan force-pushed the st3fan/Bug1225224PromptOnClearSyncedHistory branch from 011f2ff to 6179f64 Compare December 17, 2015 20:42
@st3fan st3fan force-pushed the st3fan/Bug1225224PromptOnClearSyncedHistory branch from 59134c5 to 4436204 Compare December 17, 2015 20:48
st3fan added a commit that referenced this pull request Dec 17, 2015
…yncedHistory

Fixes 1225224 - Prompt on clearing History when signed in to Sync
@st3fan st3fan merged commit 6a54991 into master Dec 17, 2015
@rnewman
Copy link
Contributor

rnewman commented Dec 18, 2015

Follow-up is #1370.

@rnewman rnewman deleted the st3fan/Bug1225224PromptOnClearSyncedHistory branch December 18, 2015 06:13
@rnewman rnewman added this to the Firefox iOS 2.0 milestone Dec 18, 2015
isabelrios pushed a commit to isabelrios/firefox-ios that referenced this pull request Feb 19, 2024
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