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.

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

Macro: The full screen button is in a place that doesn't match the design #3559

Closed
Zhirnoff opened this issue Nov 9, 2023 · 1 comment · Fixed by #3605
Closed

Macro: The full screen button is in a place that doesn't match the design #3559

Zhirnoff opened this issue Nov 9, 2023 · 1 comment · Fixed by #3605
Assignees
@chgayane
Labels
bug Macromolecules Priority: Low
Milestone
Macromolecules Mi...

Comments

@Zhirnoff
Copy link
Collaborator

Zhirnoff commented Nov 9, 2023

Steps to Reproduce

  1. Switch to Macromolecule mode
  2. Look at right bottom of application

Actual behavior
The full screen button is in a place that doesn't match the design
2023-11-09_17h14_41

Expected behavior
The full screen button is in a place that match the design
2023-11-09_17h13_59

Desktop (please complete the following information):

  • OS: Windows 10
  • Browser Chrome
  • Version 112.0.5615.138 (Official Build) (64-bit)

Ketcher version
Version 2.15.0-rc.4
@Zhirnoff Zhirnoff added this to the Macromolecules Milestone 3 milestone Nov 9, 2023
@chgayane chgayane assigned chgayane and unassigned rrodionov91 Nov 21, 2023
chgayane added a commit that referenced this issue Nov 22, 2023
@chgayane
#3559 - full screen button according to design
add36e6
@chgayane chgayane linked a pull request Nov 22, 2023 that will close this issue
Macro : #3559 - The full screen button is in a place that doesn't match the design #3605
Merged
9 tasks
chgayane added a commit that referenced this issue Nov 22, 2023
@chgayane
#3559 - full screen button back on Editor component
9483542
chgayane added a commit that referenced this issue Nov 22, 2023
@chgayane
#3559 - removed unnecessary declarations
f41fd10
chgayane added a commit that referenced this issue Nov 22, 2023
@chgayane
#3559 - updated screenshots
77298d9
chgayane added a commit that referenced this issue Nov 23, 2023
@chgayane
#3559 - style changes
61851d7
chgayane added a commit that referenced this issue Nov 23, 2023
@chgayane
#3559 - updated screenshots
198da58
chgayane added a commit that referenced this issue Nov 23, 2023
@chgayane
#3559 - styles changes and updated screenshots
fb9d761
chgayane added a commit that referenced this issue Nov 24, 2023
@chgayane
Merge pull request #3605 from epam/3559-macro-the-full-screen-button-…
d4d3d8d 
…is-in-a-place-that-doesnt-match-the-design

Macro : #3559 - The full screen button is in a place that doesn't match the design
captain2b pushed a commit that referenced this issue Nov 28, 2023
@chgayane
#3559 - full screen button according to design
3f01f09
captain2b pushed a commit that referenced this issue Nov 28, 2023
@chgayane
#3559 - full screen button back on Editor component
1177ff1
captain2b pushed a commit that referenced this issue Nov 28, 2023
@chgayane
#3559 - removed unnecessary declarations
64c9263
captain2b pushed a commit that referenced this issue Nov 28, 2023
@chgayane
#3559 - updated screenshots
9b34e3d
captain2b pushed a commit that referenced this issue Nov 28, 2023
@chgayane
#3559 - style changes
b41b569
captain2b pushed a commit that referenced this issue Nov 28, 2023
@chgayane
#3559 - updated screenshots
ff4a6e1
captain2b pushed a commit that referenced this issue Nov 28, 2023
@chgayane
#3559 - styles changes and updated screenshots
1f9c91a
@Zhirnoff
Copy link
Collaborator Author

Tested.

@github-project-automation github-project-automation bot moved this to Done in Ketcher Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chgayane chgayane

Labels
bug Macromolecules Priority: Low
Projects
Ketcher
Status: Done
Macromolecules Milestone 3
Status: Done
Milestone
Macromolecules Milestone 3
Development

Successfully merging a pull request may close this issue.

Macro : #3559 - The full screen button is in a place that doesn't match the design epam/ketcher
4 participants
@rrodionov91 @AlexeyGirin @Zhirnoff @chgayane