Skip to content

Update imageviewer.js#15730

Merged
AUTOMATIC1111 merged 1 commit intoAUTOMATIC1111:devfrom
bluelovers:patch-2
Jun 8, 2024
Merged

Update imageviewer.js#15730
AUTOMATIC1111 merged 1 commit intoAUTOMATIC1111:devfrom
bluelovers:patch-2

Conversation

@bluelovers
Copy link
Copy Markdown
Contributor

Description

  • a simple description of what you're trying to accomplish
  • a summary of changes in code
  • which issues it fixes, if any

Screenshots/videos:

Checklist:

@bluelovers bluelovers requested a review from AUTOMATIC1111 as a code owner May 7, 2024 11:53
@bluelovers bluelovers changed the base branch from master to dev May 7, 2024 12:00
@w-e-w
Copy link
Copy Markdown
Collaborator

w-e-w commented May 7, 2024

Description

  • a simple description of what you're trying to accomplish
  • a summary of changes in code
  • which issues it fixes, if any

@light-and-ray
Copy link
Copy Markdown
Contributor

What happens here? 🤨

@w-e-w
Copy link
Copy Markdown
Collaborator

w-e-w commented May 7, 2024

https://github.com/AUTOMATIC1111/stable-diffusion-webui/pulls?q=is%3Apr+author%3Abluelovers+
this guy pretty much never properly writes PR descriptions
at most a couple of screenshot or one line description

@light-and-ray
Copy link
Copy Markdown
Contributor

light-and-ray commented May 7, 2024

I know there is a bug in full screen image view on mobile phones. If there are more then one image, the next image is opened every time. Maybe this pr is about it. Or if not, it's needed to fix btw

@bluelovers
Copy link
Copy Markdown
Contributor Author

image

@AUTOMATIC1111
Copy link
Copy Markdown
Owner

i think doesn't fix anything but just makes the code smaller

@AUTOMATIC1111 AUTOMATIC1111 merged commit 4aebfe9 into AUTOMATIC1111:dev Jun 8, 2024
@bluelovers bluelovers deleted the patch-2 branch July 7, 2024 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants