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

Improve Header bar #2441

Merged
merged 1 commit into from
Jul 20, 2022
Merged

Improve Header bar #2441

merged 1 commit into from
Jul 20, 2022

Conversation

luka-nextcloud
Copy link
Contributor

@luka-nextcloud luka-nextcloud commented May 24, 2022

Signed-off-by: Luka Trovic luka@nextcloud.com

Summary

Some improvements on the header bar.

@luka-nextcloud luka-nextcloud self-assigned this May 24, 2022
@luka-nextcloud luka-nextcloud changed the title feat: update header bar Improve Header bar May 24, 2022
css/style.scss Outdated Show resolved Hide resolved
@jancborchardt
Copy link
Member

Before/after screenshots would be great for quick design review @luka-nextcloud @vinicius73 :) thanks!

@luka-nextcloud
Copy link
Contributor Author

Before/after screenshots would be great for quick design review @luka-nextcloud @vinicius73 :) thanks!

@jancborchardt Please check the following screenshots.

Before:
image

After:
image

@luka-nextcloud luka-nextcloud force-pushed the feature/2440-improve-header-bar branch from e94308d to af81b2e Compare June 7, 2022 14:08
@juliusknorr juliusknorr added this to the Nextcloud 25 milestone Jun 9, 2022
@luka-nextcloud luka-nextcloud force-pushed the feature/2440-improve-header-bar branch from af81b2e to 79284bc Compare June 21, 2022 07:42
Copy link
Member

@jancborchardt jancborchardt left a comment

Choose a reason for hiding this comment

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

Looks great! :)

@vinicius73
Copy link
Member

/rebase

@nextcloud-command nextcloud-command force-pushed the feature/2440-improve-header-bar branch from 79284bc to d57c620 Compare June 30, 2022 14:22
@luka-nextcloud luka-nextcloud force-pushed the feature/2440-improve-header-bar branch 2 times, most recently from b243c4d to d2cdf75 Compare July 15, 2022 17:04
@vinicius73
Copy link
Member

/rebase

@nextcloud-command nextcloud-command force-pushed the feature/2440-improve-header-bar branch from d2cdf75 to 3bcf161 Compare July 19, 2022 14:54
Copy link
Member

@juliusknorr juliusknorr left a comment

Choose a reason for hiding this comment

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

@vinicius73 Let's wait for the merge of the viewer PR, as this one then needs to be adjusted to use the data attribute.

@luka-nextcloud luka-nextcloud force-pushed the feature/2440-improve-header-bar branch from 3bcf161 to f507878 Compare July 20, 2022 05:31
@juliusknorr
Copy link
Member

/compile amend /

Signed-off-by: Luka Trovic <luka@nextcloud.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@nextcloud-command nextcloud-command force-pushed the feature/2440-improve-header-bar branch from f507878 to 0b130d0 Compare July 20, 2022 05:58
@juliusknorr juliusknorr merged commit 41c7b82 into master Jul 20, 2022
@delete-merged-branch delete-merged-branch bot deleted the feature/2440-improve-header-bar branch July 20, 2022 06:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve Header bar
6 participants