Skip to content

Conversation

@bluelovers
Copy link
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
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
Contributor

What happens here? 🤨

@w-e-w
Copy link
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
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
Contributor Author

image

@AUTOMATIC1111
Copy link
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