-
Notifications
You must be signed in to change notification settings - Fork 3
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
Toolbar and omnibar changes #18
Conversation
MasterK0927
commented
Apr 6, 2024
- Hid the Brave_action icons i.e. brave_shield and brave_rewards.
- Hid the Licensed photo author toggle from the background.
- Stopped the generation of License for the background image.
script/generate_licenses.py
Outdated
@@ -161,36 +161,36 @@ def validated_data_field(data, field_name): | |||
return field_value | |||
|
|||
|
|||
def generate_backgrounds_license(preamble, backgrounds): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we dont need to comment this out if already commented below
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
actually yes, but this code is not needed by us
ntp_backgrounds_preamble, ntp_backgrounds)): | ||
print(f'- {len(ntp_backgrounds)} sub-components added in ' \ | ||
'brave_new_tab_ui/data/LICENSE') | ||
# ntp_backgrounds = list_ntp_backgrounds( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
try to remove only the things not needed. if this is already removed from react then need not comment this
@@ -1,4 +1,4 @@ | |||
<!doctype html> | |||
<!-- <!doctype html> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is this file doing?