Skip to content

Feat/mobile mvp #588

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 18 commits into
base: main
Choose a base branch
from
Open

Feat/mobile mvp #588

wants to merge 18 commits into from
+261 −45

Conversation

iapolya
Copy link

@iapolya iapolya commented Feb 6, 2025 •
edited by makhnatkin
Loading

Fixes #329

  1. Tooltips of cuts are hidden
Screenshot 2025-02-06 at 12 29 25
  1. Additional actions and mode switching are displayed as a Sheet
Screenshot 2025-02-06 at 12 30 02 Screenshot 2025-02-06 at 12 30 53
  1. The selection menu is hidden
Screenshot 2025-02-06 at 12 31 30
  1. Part of the actions is hidden as they require design refinement

Popup list (sheet?): Action.emoji, Action.colorify
Complex popup (100% design is required) 1: Action.file, Action.image, Action.imagePopup, Action.link,
Complex actions: Action.table, Action.filePopup
Under consideration: Action.horizontalRule

@gravity-ui-bot
Copy link
Contributor

Preview is ready.

@d3m1d0v d3m1d0v changed the base branch from main to v15next February 6, 2025 09:40
@d3m1d0v
Copy link
Member

d3m1d0v commented Feb 6, 2025

I changed target branch to v15next because source branch contains commits from v15next

I suggest reviewing and merging this PR after we migrating to uikit7

@iapolya iapolya force-pushed the feat/mobile-mvp branch 2 times, most recently from 05c6811 to b98ca42 Compare February 6, 2025 11:05
Base automatically changed from v15next to main February 11, 2025 12:28
@iapolya iapolya force-pushed the feat/mobile-mvp branch 4 times, most recently from 236d26f to bedb3ff Compare February 12, 2025 08:14
makhnatkin
makhnatkin reviewed Feb 18, 2025
View reviewed changes
@iapolya iapolya marked this pull request as draft February 18, 2025 16:01
@iapolya
Copy link
Author

iapolya commented May 17, 2025 •
edited
Loading

TODO:

  1. Remove list
Screenshot 2025-05-17 at 16 20 14

@iapolya iapolya force-pushed the feat/mobile-mvp branch from bedb3ff to a3098b4 Compare May 17, 2025 15:59
@gravity-ui-bot
Copy link
Contributor

Visual Tests Report is ready.

@iapolya iapolya force-pushed the feat/mobile-mvp branch from f1aca3f to 254cf48 Compare May 17, 2025 16:31
@iapolya iapolya force-pushed the feat/mobile-mvp branch from 254cf48 to aba0449 Compare June 6, 2025 13:20
@iapolya iapolya force-pushed the feat/mobile-mvp branch from aba0449 to e27fab3 Compare June 6, 2025 13:23
@iapolya iapolya force-pushed the feat/mobile-mvp branch from e27fab3 to 53bd29b Compare June 6, 2025 13:27
@iapolya iapolya marked this pull request as ready for review June 6, 2025 14:17
@iapolya iapolya force-pushed the feat/mobile-mvp branch from 974ce51 to 74e2341 Compare June 6, 2025 14:20
@iapolya iapolya force-pushed the feat/mobile-mvp branch from 74e2341 to b95e83a Compare June 6, 2025 14:24
@iapolya iapolya requested a review from makhnatkin June 25, 2025 20:20
makhnatkin
makhnatkin reviewed Jun 27, 2025
View reviewed changes
@iapolya iapolya requested a review from makhnatkin July 15, 2025 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@d3m1d0v d3m1d0v Awaiting requested review from d3m1d0v d3m1d0v is a code owner

@makhnatkin makhnatkin Awaiting requested review from makhnatkin makhnatkin is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Mobile view
4 participants
@iapolya @gravity-ui-bot @d3m1d0v @makhnatkin