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

special display case for logged-out users to display download in watch page #2919

Merged
merged 1 commit into from
Aug 4, 2020

Conversation

rigelk
Copy link
Collaborator

@rigelk rigelk commented Jun 30, 2020

No description provided.

@rigelk rigelk added the UI non-trivial UI changes, that might need discussion label Jun 30, 2020
@rigelk rigelk linked an issue Jun 30, 2020 that may be closed by this pull request
@rigelk rigelk requested a review from Chocobozzz June 30, 2020 16:00
@Chocobozzz
Copy link
Owner

Put this logic in video watch component instead, because we only want this behaviour in this specific page

@rigelk rigelk merged commit 6863f81 into Chocobozzz:develop Aug 4, 2020
@rigelk rigelk deleted the download-button branch August 4, 2020 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI non-trivial UI changes, that might need discussion
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Why hiding download button behind a "..." button ? UI
2 participants