Combine two VBAs, Auto Resize Image and then Center Align it

[Jeremy Zhang]

Hi,

I am new to VBA, I have two VBA codes from the web and they work fine for me except below:

• For the auto resize image, I would like to set the image size a little bit smaller than the cells
• I woud also like to combine these two codes to one, so that I can complete these two actiosn by one time

Thank you!

Public Sub AutoResizeImage()
On Error GoTo Select_Image
Dim ZImageWtoHRatio As Single
Dim QWtoHRatio As Single
With Selection
ZImageWtoHRatio = .Width / .Height
End With
With Selection.TopLeftCell
QWtoHRatio = .Width / .RowHeight
End With
Select Case ZImageWtoHRatio / QWtoHRatio
Case Is > 1
With Selection
.Width = .TopLeftCell.Width
.Height = .Width / ZImageWtoHRatio
End With
Case Else
With Selection
.Height = .TopLeftCell.RowHeight
.Width = .Height * ZImageWtoHRatio
End With
End Select
With Selection
.Top = .TopLeftCell.Top
.Left = .TopLeftCell.Left
End With
Exit Sub
Select_Image:
MsgBox "Choose an Image and Run the Macro."
End Sub

Sub CenterPicturesHnV()

    Dim myR As Range
    Dim myP As Shape

    For Each myP In ActiveSheet.Shapes
        Set myR = myP.TopLeftCell
        myP.Left = myR.Left + (myR.Width - myP.Width) / 2
        myP.Top = myR.Top + (myR.Height - myP.Height) / 2
     Next myP
End Sub

 

[nemmi69]

Combined

Public Sub AutoResizeImage()
On Error GoTo Select_Image
Dim ZImageWtoHRatio As Single
Dim QWtoHRatio As Single
Dim myR As Range
Dim myP As Shape
With Selection
    ZImageWtoHRatio = .Width / .Height
End With
With Selection.TopLeftCell
    QWtoHRatio = .Width / .RowHeight
End With
Select Case ZImageWtoHRatio / QWtoHRatio
Case Is > 1
    With Selection
    .Width = .TopLeftCell.Width
    .Height = .Width / ZImageWtoHRatio
    End With
Case Else
    With Selection
    .Height = .TopLeftCell.RowHeight
    .Width = .Height * ZImageWtoHRatio
    End With
End Select
With Selection
    .Top = .TopLeftCell.Top
    .Left = .TopLeftCell.Left
End With
' Center Pictures
For Each myP In ActiveSheet.Shapes
    Set myR = myP.TopLeftCell
    myP.Left = myR.Left + (myR.Width - myP.Width) / 2 '## try playing with the /2 maybe 2.25
    myP.Top = myR.Top + (myR.Height - myP.Height) / 2
Next myP
Exit Sub

Select_Image:
    MsgBox "Choose an Image and Run the Macro."
End Sub

 

[Jeremy Zhang]

Combined

Public Sub AutoResizeImage()
On Error GoTo Select_Image
Dim ZImageWtoHRatio As Single
Dim QWtoHRatio As Single
Dim myR As Range
Dim myP As Shape
With Selection
    ZImageWtoHRatio = .Width / .Height
End With
With Selection.TopLeftCell
    QWtoHRatio = .Width / .RowHeight
End With
Select Case ZImageWtoHRatio / QWtoHRatio
Case Is > 1
    With Selection
    .Width = .TopLeftCell.Width
    .Height = .Width / ZImageWtoHRatio
    End With
Case Else
    With Selection
    .Height = .TopLeftCell.RowHeight
    .Width = .Height * ZImageWtoHRatio
    End With
End Select
With Selection
    .Top = .TopLeftCell.Top
    .Left = .TopLeftCell.Left
End With
' Center Pictures
For Each myP In ActiveSheet.Shapes
    Set myR = myP.TopLeftCell
    myP.Left = myR.Left + (myR.Width - myP.Width) / 2 '## try playing with the /2 maybe 2.25
    myP.Top = myR.Top + (myR.Height - myP.Height) / 2
Next myP
Exit Sub

Select_Image:
    MsgBox "Choose an Image and Run the Macro."
End Sub

Thank you so much! The combined codes work for most images, but sometimes the image may go to another cell during the centerring process, did you see any issues in the code on image properties etc.?

 

[nemmi69]

Just noticed I referred to the wrong part for sizing. This affects the size.

With Selection
    ZImageWtoHRatio = .Width / .Height
End With

 

[Jeremy Zhang]

Just noticed I referred to the wrong part for sizing. This affects the size.

With Selection
    ZImageWtoHRatio = .Width / .Height
End With

Solved now, thank you so much!!!

 

[nemmi69]

No problem