Photo Email

bouncey

New Member
Joined
Jan 13, 2010
Messages
35
H There, I have the following code that looks up a photo based on column A and places the photo in column L. The problem is when I email the file the photo's are not emailed. Is there any way to fix the photo's?

Code:
Private Sub Worksheet_Change(ByVal Target As Range)
 
Dim myPict As Picture
Dim PictureLoc As String
Dim lr As Long, r As Long
lr = Cells(Rows.Count, "A").End(xlUp).Row
 
ActiveSheet.Pictures.Delete
For r = 2 To lr
PictureLoc = "C\photography\" & Range("A" & r).Value & ".jpg"
 
With Range("l" & r)
    Set myPict = ActiveSheet.Pictures.AddPicture(PictureLoc)
    .RowHeight = 144
    myPict.Top = .Top
    myPict.Left = .Left
    myPict.Width = 110
    myPict.Height = 140
   
    
    myPict.Placement = xlMoveAndSize
End With
 
Next r
End Sub

Thanks in advance.
 

Excel Facts

Why does 9 mean SUM in SUBTOTAL?
It is because Sum is the 9th alphabetically in Average, Count, CountA, Max, Min, Product, StDev.S, StDev.P, Sum, VAR.S, VAR.P.
.
If you are open to using a different macro , this works here after emailing.

Code:
Option Explicit


Sub AddOlEObject()


    Dim mainWorkBook As Workbook
    Dim Folderpath, fStr, myPath, Filename As String
    Dim fso, NoOfFiles, listfiles, fls, strCompFilePath
    Dim counter
    
    Set mainWorkBook = ActiveWorkbook
    Sheets("Sheet1").Activate
    Folderpath = "C:\Users\My\Desktop\Pics"   '<-- Change path to your images here
    Set fso = CreateObject("Scripting.FileSystemObject")
    NoOfFiles = fso.GetFolder(Folderpath).Files.Count
    Set listfiles = fso.GetFolder(Folderpath).Files
    For Each fls In listfiles
       strCompFilePath = Folderpath & "\" & Trim(fls.Name)
        If strCompFilePath <> "" Then
        
            '// include image extensions here \\
            
            If (InStr(1, strCompFilePath, "jpg", vbTextCompare) > 1 _
            Or InStr(1, strCompFilePath, "jpeg", vbTextCompare) > 1 _
            Or InStr(1, strCompFilePath, "png", vbTextCompare) > 1) Then
                 counter = counter + 1
                 Filename = fls.Name
                    If InStr(Filename, ".") > 0 Then
                       Filename = Left(Filename, InStr(Filename, ".") - 1)
                    End If
                  Sheets("Sheet1").Range("A" & counter).Value = Filename
                  Sheets("Sheet1").Range("B" & counter).ColumnWidth = 25
                Sheets("Sheet1").Range("B" & counter).RowHeight = 100
                Sheets("Sheet1").Range("B" & counter).Activate
                Call insert(strCompFilePath, counter)
                Sheets("Sheet1").Activate
            End If
        End If
    Next
'mainWorkBook.Save




End Sub


Function insert(PicPath, counter)
'MsgBox PicPath
    With ActiveSheet.Pictures.insert(PicPath)
    
    '// change image sizes here \\
        
        With .ShapeRange
            .LockAspectRatio = msoTrue
            .Width = 50
            .Height = 70
        End With
        .Left = ActiveSheet.Range("B" & counter).Left
        .Top = ActiveSheet.Range("B" & counter).Top
        .Placement = 1
        .PrintObject = True
    End With
End Function
 
Upvote 0
.
Be certain you edit the path to the location of your images :

Code:
[COLOR=#333333]Folderpath = "C:\Users\My\Desktop\Pics"   '<-- Change path to your images here[/COLOR]

My computer is named "My" and the images/photos I was working with were located on the desktop in a folder named "Pics".

You will need to edit the path to direct the macro to the location of your images.
 
Upvote 0
thanks Logit - this works on a different basis to what I am trying to do. I need to enter a code in to a cell in column a which then looks up the correct photo and enter into column l. Your sheet enters all photos in the file.
 
Upvote 0
.
I was re-reading your original post and finally realized I misunderstood your goal. Accept my apologies.

You want to copy images in a range and have them pasted to the body of an email. This macro will do that :

Code:
Option Explicit




Sub mailpicsorchart()
Dim OutApp As Object
Dim OutMail As Object
Dim vInspector, GetInspector, wEditor As Variant




Set OutApp = CreateObject("Outlook.Application")
Set OutMail = OutApp.CreateItem(0)
With OutMail
    .To = "yo momma@nowhere.com"
    .CC = "xyz@anc.com"
    .BCC = "abc@xyz.com"
    .Subject = "Test"
    .Body = "Dear" & "Macro " & vbCrLf
    .Display
    ActiveSheet.Range("K21:K26").Copy   '<-- range must cover all cells where pics are pasted
    Set vInspector = OutMail.GetInspector
    Set wEditor = vInspector.WordEditor


    wEditor.Application.Selection.Start = Len(.Body)
    wEditor.Application.Selection.End = wEditor.Application.Selection.Start


    wEditor.Application.Selection.Paste


.Display
End With
End Sub

Note the comment that the stated range in the macro must cover all cells where the images are pasted. You can include a column more on
either side of the pic/s and a column more above and below all the pic/s, it won't disrupt the process.

Hope I got it right this time. :rolleyes:
 
Upvote 0

Forum statistics

Threads
1,214,583
Messages
6,120,380
Members
448,955
Latest member
BatCoder

We've detected that you are using an adblocker.

We have a great community of people providing Excel help here, but the hosting costs are enormous. You can help keep this site running by allowing ads on MrExcel.com.
Allow Ads at MrExcel

Which adblocker are you using?

Disable AdBlock

Follow these easy steps to disable AdBlock

1)Click on the icon in the browser’s toolbar.
2)Click on the icon in the browser’s toolbar.
2)Click on the "Pause on this site" option.
Go back

Disable AdBlock Plus

Follow these easy steps to disable AdBlock Plus

1)Click on the icon in the browser’s toolbar.
2)Click on the toggle to disable it for "mrexcel.com".
Go back

Disable uBlock Origin

Follow these easy steps to disable uBlock Origin

1)Click on the icon in the browser’s toolbar.
2)Click on the "Power" button.
3)Click on the "Refresh" button.
Go back

Disable uBlock

Follow these easy steps to disable uBlock

1)Click on the icon in the browser’s toolbar.
2)Click on the "Power" button.
3)Click on the "Refresh" button.
Go back
Back
Top