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

Adjust footer height #30586

Closed
wants to merge 2 commits into from
Closed

Adjust footer height #30586

wants to merge 2 commits into from

Conversation

solracsf
Copy link
Member

65px is not enough when you have 3 lines (Cloud name, Legal notice and Signup link)

image

@solracsf solracsf added design Design, UI, UX, etc. 3. to review Waiting for reviews labels Jan 11, 2022
@solracsf
Copy link
Member Author

/backport to stable23

@solracsf
Copy link
Member Author

/backport to stable22

@artonge
Copy link
Contributor

artonge commented Jan 13, 2022

Feels like 100px is then too big for only two lines.

@jancborchardt an opinion on this one? It is about the footer at the bottom of public shares and other public views.

@solracsf
Copy link
Member Author

@artonge 80px indeed fixes it too. Maybe its a compromise in between?

@CarlSchwan

This comment was marked as resolved.

@solracsf solracsf closed this Jan 13, 2022
@solracsf solracsf deleted the footer-height branch January 13, 2022 18:28
@solracsf solracsf restored the footer-height branch February 28, 2022 19:27
@solracsf solracsf reopened this Feb 28, 2022
@solracsf solracsf added this to the Nextcloud 24 milestone Feb 28, 2022
@szaimen
Copy link
Contributor

szaimen commented Mar 1, 2022

/rebase

@skjnldsv skjnldsv mentioned this pull request Mar 24, 2022
@blizzz blizzz mentioned this pull request Mar 31, 2022
@blizzz blizzz mentioned this pull request Apr 7, 2022
@szaimen
Copy link
Contributor

szaimen commented Apr 7, 2022

/rebase

@szaimen szaimen requested review from a team, PVince81, artonge and Pytal and removed request for a team April 7, 2022 15:05
@szaimen szaimen added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Apr 7, 2022
@solracsf

This comment was marked as resolved.

@blizzz blizzz mentioned this pull request Apr 13, 2022
@blizzz blizzz modified the milestones: Nextcloud 24, Nextcloud 25 Apr 21, 2022
@solracsf
Copy link
Member Author

/backport to stable24

@szaimen
Copy link
Contributor

szaimen commented May 27, 2022

@solracsf can you please squash? Then we can merge :)

@solracsf
Copy link
Member Author

🌝

image

@szaimen
Copy link
Contributor

szaimen commented May 27, 2022

Screenshot_20220527-133938_Brave

@solracsf
Copy link
Member Author

Maybe @blizzz can enable such (very useful) option?

@blizzz
Copy link
Member

blizzz commented Jun 21, 2022

Maybe @blizzz can enable such (very useful) option?

it would result in a fast forward merge, and this is not what we went when merging into master or stable branches.

This was referenced Aug 12, 2022
This was referenced Aug 24, 2022
@blizzz blizzz mentioned this pull request Sep 6, 2022
@solracsf
Copy link
Member Author

solracsf commented Sep 6, 2022

/rebase

65px is not enough when you have 3 lines (Cloud name, Legal notice and Signup link)
@blizzz blizzz mentioned this pull request Sep 9, 2022
@skjnldsv skjnldsv mentioned this pull request Sep 15, 2022
This was referenced Sep 20, 2022
@blizzz blizzz modified the milestones: Nextcloud 25, Nextcloud 26 Sep 22, 2022
@solracsf
Copy link
Member Author

solracsf commented Oct 3, 2022

Closing and moving to #34391

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish design Design, UI, UX, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants