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

[*]BO: New navigation & header #4822

Merged
merged 25 commits into from Feb 1, 2016

Conversation

@Shudrum
Copy link
Contributor

commented Jan 29, 2016

The new tabs & header are now here!

Some things are now broken and will be up ASAP:

  • The element where modules icons are placed on the header will be put at the right place
  • All the notifications will be put on the new single notifications center
  • The loader will be different
  • The multishop select will be put again
@@ -1854,6 +1854,7 @@ CREATE TABLE `PREFIX_tab` (
`position` int(10) unsigned NOT NULL,
`active` tinyint(1) NOT NULL DEFAULT 1,
`hide_host_mode` tinyint(1) NOT NULL DEFAULT '0',
`icon` varchar(32) DEFAULT '',

This comment has been minimized.

Copy link
@xGouley

xGouley Jan 29, 2016

Member

report this in upgrade sql fil (1.7.0.0.sql) ?

This comment has been minimized.

Copy link
@Shudrum

Shudrum Jan 29, 2016

Author Contributor

When the final navigation will be finished (maybe some things will move), the upgrade will be done for this, and all the modifications done on the xml files.

@@ -2589,6 +2586,7 @@ public function setMedia()
//Bootstrap
$this->addCSS(__PS_BASE_URI__.$this->admin_webpath.'/themes/'.$this->bo_theme.'/css/'.$this->bo_css, 'all', 0);
$this->addCSS(__PS_BASE_URI__.$this->admin_webpath.'/themes/'.$this->bo_theme.'/css/vendor/titatoggle-min.css', 'all', 0);
$this->addCSS('https://fonts.googleapis.com/icon?family=Material+Icons', 'all', 0);

This comment has been minimized.

Copy link
@xGouley

xGouley Jan 29, 2016

Member

Question:
http / https ? both works, why https ?

This comment has been minimized.

Copy link
@Shudrum

Shudrum Jan 29, 2016

Author Contributor

Because this line will be removed when the UI Kit will come. This is temporary.

@xGouley

This comment has been minimized.

Copy link
Member

commented Jan 29, 2016

👍 for me except sass part not reviewed.

@Shudrum

This comment has been minimized.

Copy link
Contributor Author

commented Jan 29, 2016

Sass is as clean as before 😷
All will be re-indented soon.

Thank you !

xGouley added a commit that referenced this pull request Feb 1, 2016
Merge pull request #4822 from Shudrum/navigation
[*]BO: New navigation & header

@xGouley xGouley merged commit 74d488b into PrestaShop:develop Feb 1, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@Shudrum

This comment has been minimized.

Copy link
Contributor Author

commented Feb 1, 2016

Thank you!

@xGouley

This comment has been minimized.

Copy link
Member

commented Feb 1, 2016

You are welcome! 💐

@Shudrum Shudrum deleted the Shudrum:navigation branch Feb 23, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.