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

Rebranded Private Page and other changes #11

Closed
wants to merge 13 commits into from

Conversation

MasterK0927
Copy link
Collaborator

  1. Rebranded the UI of Private aka Incognito Page
  2. Changed the background image of new tab
  3. Removed the "Scroll for Brave News" prompt from the New Tab page
  4. Changed Brave -> Ping in the strings file responsible for private tab texts
  5. Changed the default state of SideBar to NEVER i.e. hidden it
  6. Removed the SideBar customisation settings from the settings/appearance tab
  7. Changed the default state of Big Search bar to true
  8. Set the default state of ShowSideBar Button to false

Copy link
Collaborator

Choose a reason for hiding this comment

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

these changes were also already merged. why are they showing up again.

Try creating a branch from this repo itself instead the fork and then raise a pr

Copy link
Collaborator

Choose a reason for hiding this comment

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

ye ek icon delete karne se fix nahi hoga. tumhari pichli pr changes isme firse aa rahe hai. jo changes already merge ho gaye wo firse nahi ane chaiye na

Copy link
Collaborator

Choose a reason for hiding this comment

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

ye bhi toh last pr mein change kiya tha

@@ -49,31 +49,31 @@
</message>

<message name="IDS_BRAVE_WELCOME_SET_DEFAULT_BUTTON_LABEL" desc="A button that sets Brave browser as default">
Set Brave as default browser
Set Ping as default browser
Copy link
Collaborator

Choose a reason for hiding this comment

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

ye sab bhi

@@ -7,7 +7,7 @@ import { provideStrings } from '../../../.storybook/locale'

provideStrings({
braveWelcomeTitle: 'Privacy. By Default.',
braveWelcomeDesc: 'Get Brave protection on every link you click. Just set Brave as default and browse. Privately.',
braveWelcomeDesc: 'Get Ping protection on every link you click. Just set Ping as default and browse. Privately.',
Copy link
Collaborator

Choose a reason for hiding this comment

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

this as well

@MasterK0927 MasterK0927 deleted the keshav branch March 29, 2024 13:54
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