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

polishing of sidebar in public view #12132

Closed
2 tasks done
szaimen opened this issue Mar 11, 2024 · 7 comments · Fixed by #12133 or #12825
Closed
2 tasks done

polishing of sidebar in public view #12132

szaimen opened this issue Mar 11, 2024 · 7 comments · Fixed by #12133 or #12825
Assignees
Labels
1. to develop bug design feature: frontend 🖌️ "Web UI" client feature: talk-sidebar ⬅️ Sidebar integration of Talk into other apps like sharing and documents
Milestone

Comments

@szaimen
Copy link
Contributor

szaimen commented Mar 11, 2024

just tested on nc29.0.0b1

@Pytal Pytal transferred this issue from nextcloud/server Apr 16, 2024
@Pytal
Copy link
Member

Pytal commented Apr 16, 2024

Seems to be the Talk public share sidebar, cc @danxuliu

@nickvergessen nickvergessen added feature: frontend 🖌️ "Web UI" client feature: talk-sidebar ⬅️ Sidebar integration of Talk into other apps like sharing and documents labels Apr 17, 2024
@nickvergessen nickvergessen added this to the 💞 Next RC (29) milestone Apr 17, 2024
@github-project-automation github-project-automation bot moved this from 🕶️ Design review to 🎉 Done in 🖍 Design team Apr 17, 2024
@nickvergessen
Copy link
Member

Reopening as the icon color and file name color is still wrong if I see that correctly

@nickvergessen nickvergessen reopened this Apr 17, 2024
@nickvergessen
Copy link
Member

And just for the record, we need to adjust the used variable, not the content of the variable. it claims to be a button tertiary-on-primary, but it's not on primary, it's on the background.
Might be non trivial to solve. Best is to check the color/variable used by the app list when you are logged in

@Antreesy
Copy link
Contributor

That's the result of applying tertiary-no-background. But I'm not sure it's consistent:

  • either both NcButton and NcActions here should have dark color on a dark theme
  • or file name and "Download" button should have white color

image

@jancborchardt jancborchardt moved this from 🎉 Done to 🏗️ At engineering in 🖍 Design team Apr 17, 2024
@susnux
Copy link

susnux commented Jul 26, 2024

should be fixed with:
nextcloud/server#46785

@susnux susnux closed this as completed Jul 26, 2024
@github-project-automation github-project-automation bot moved this from 🏗️ At engineering to 🎉 Done in 🖍 Design team Jul 26, 2024
@susnux
Copy link

susnux commented Jul 26, 2024

correct color is btw: --color-background-plain-text

@Antreesy
Copy link
Contributor

Nein, that indeed fixes the file name, but not the button =/

image

It's Talk-only now, so I'll take over from here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop bug design feature: frontend 🖌️ "Web UI" client feature: talk-sidebar ⬅️ Sidebar integration of Talk into other apps like sharing and documents
Projects
Archived in project
6 participants