Skip to content
This repository has been archived by the owner. It is now read-only.

Native ActionSheet implementation for 3dots menu, and MyOffrz integration #12

Merged
merged 3 commits into from Mar 22, 2018

Conversation

@naira-cliqz
Copy link
Contributor

@naira-cliqz naira-cliqz commented Mar 20, 2018

No description provided.

@@ -347,6 +347,7 @@ class BrowserViewController: UIViewController {
urlBar = URLBarView()
*/
urlBar = CliqzURLBar()
NotificationCenter.default.addObserver(self, selector: #selector(urlBarDidPressPageCliqzOptions), name: URLBarDidPressPageOptionsNotification, object: nil)

This comment has been minimized.

@mahmoud-adam85

mahmoud-adam85 Mar 22, 2018
Contributor

could you fix the indentation of this line

This comment has been minimized.

@naira-cliqz

naira-cliqz Mar 22, 2018
Author Contributor

The indentation is fine, github messes up...

@@ -145,6 +155,9 @@ class CliqzURLBar: URLBarView {
}

}

override func tabLocationViewDidTapPageOptions(_ tabLocationView: TabLocationView, from button: UIButton) { self.delegate?.urlBarDidPressCliqzPageOptions(self, from: tabLocationView.pageOptionsButton)

This comment has been minimized.

@mahmoud-adam85

mahmoud-adam85 Mar 22, 2018
Contributor

could you add new line to method body

This comment has been minimized.

@naira-cliqz

naira-cliqz Mar 22, 2018
Author Contributor

This one is fixed

@naira-cliqz naira-cliqz force-pushed the naira-cliqz:master branch from cedd1fb to fac4fbd Mar 22, 2018
@naira-cliqz naira-cliqz reopened this Mar 22, 2018
@naira-cliqz naira-cliqz force-pushed the naira-cliqz:master branch from fac4fbd to a0296fa Mar 22, 2018
@mahmoud-adam85 mahmoud-adam85 changed the title Native ActionSheet implementation for 3dots menu Native ActionSheet implementation for 3dots menu, and MyOffrz integration Mar 22, 2018
@mahmoud-adam85 mahmoud-adam85 merged commit 0a9e552 into ghostery:master Mar 22, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants