Update app.py

app.py

@@ -313,14 +313,26 @@ class CertificateGenerator:
         return fonts
     
     def _add_professional_border(self, draw: ImageDraw.Draw):
-        # Single elegant inner border with padding
+        """Add professional border with improved corners"""
         padding = 40
-        draw.rectangle(
-            [(padding, padding), 
-             (self.certificate_size[0] - padding, self.certificate_size[1] - padding)],
-            outline='#1C1D1F',
-            width=2
-        )
+        border_width = 2
+        corner_radius = 10
+        
+        # Draw rounded rectangle border
+        x0, y0 = padding, padding
+        x1, y1 = self.certificate_size[0] - padding, self.certificate_size[1] - padding
+        
+        # Draw corners
+        draw.arc((x0, y0, x0 + corner_radius * 2, y0 + corner_radius * 2), 180, 270, '#1C1D1F', border_width)
+        draw.arc((x1 - corner_radius * 2, y0, x1, y0 + corner_radius * 2), 270, 0, '#1C1D1F', border_width)
+        draw.arc((x0, y1 - corner_radius * 2, x0 + corner_radius * 2, y1), 90, 180, '#1C1D1F', border_width)
+        draw.arc((x1 - corner_radius * 2, y1 - corner_radius * 2, x1, y1), 0, 90, '#1C1D1F', border_width)
+        
+        # Draw lines
+        draw.line((x0 + corner_radius, y0, x1 - corner_radius, y0), '#1C1D1F', border_width)  # Top
+        draw.line((x0 + corner_radius, y1, x1 - corner_radius, y1), '#1C1D1F', border_width)  # Bottom
+        draw.line((x0, y0 + corner_radius, x0, y1 - corner_radius), '#1C1D1F', border_width)  # Left
+        draw.line((x1, y0 + corner_radius, x1, y1 - corner_radius), '#1C1D1F', border_width)  # Right
     
     def _add_content(
         self,
@@ -376,23 +388,27 @@ class CertificateGenerator:
             print(f"Error adding logo: {e}")
     
     def _add_photo(self, certificate: Image.Image, photo_path: str):
-        """Add a circular profile photo centered at the top of the certificate"""
+        """Add a clear circular profile photo in the top-right corner"""
         try:
-            # Open and convert photo
+            # Open and process photo
             photo = Image.open(photo_path)
             
-            # Define size for circular photo
-            size = (120, 120)  # Slightly larger size
+            # Define size for circular photo (slightly larger for better quality)
+            size = (120, 120)
+            
+            # Convert to RGBA if not already
+            if photo.mode != 'RGBA':
+                photo = photo.convert('RGBA')
             
-            # Create circular mask
+            # Create high-quality circular mask
             mask = Image.new('L', size, 0)
             draw = ImageDraw.Draw(mask)
             draw.ellipse((0, 0, size[0], size[1]), fill=255)
             
-            # Resize photo maintaining aspect ratio and center crop
+            # Resize photo maintaining aspect ratio
             aspect = photo.width / photo.height
             if aspect > 1:
-                # Width is greater, resize based on height
+                # Width is greater
                 new_height = size[1]
                 new_width = int(new_height * aspect)
                 photo = photo.resize((new_width, new_height), Image.Resampling.LANCZOS)
@@ -400,7 +416,7 @@ class CertificateGenerator:
                 left = (new_width - size[0]) // 2
                 photo = photo.crop((left, 0, left + size[0], size[1]))
             else:
-                # Height is greater, resize based on width
+                # Height is greater
                 new_width = size[0]
                 new_height = int(new_width / aspect)
                 photo = photo.resize((new_width, new_height), Image.Resampling.LANCZOS)
@@ -411,17 +427,28 @@ class CertificateGenerator:
             # Create output image with transparency
             output = Image.new('RGBA', size, (0, 0, 0, 0))
             output.paste(photo, (0, 0))
+            
+            # Apply antialiasing to the mask
+            mask = mask.filter(ImageFilter.GaussianBlur(radius=1))
             output.putalpha(mask)
             
-            # Calculate center position for photo
-            photo_x = (certificate.width - size[0]) // 2
-            photo_y = 40  # Distance from top
+            # Position in top-right corner with padding
+            padding_right = 60
+            padding_top = 40
+            position = (certificate.width - size[0] - padding_right, padding_top)
             
-            # Paste the circular photo
-            certificate.paste(output, (photo_x, photo_y), mask=output)
+            # Add a subtle white background circle for better visibility
+            bg = Image.new('RGBA', size, (255, 255, 255, 255))
+            bg_mask = mask.copy()
+            certificate.paste(bg, position, mask=bg_mask)
             
-        except Exception as e:
-            print(f"Error adding photo: {str(e)}")
+            # Paste the photo
+            certificate.paste(output, position, mask=output)
+        
+    except Exception as e:
+        print(f"Error adding photo: {str(e)}")
+        print(f"Photo path: {photo_path}")
+        print(f"Photo details: {getattr(photo, 'size', 'unknown size')}, {getattr(photo, 'mode', 'unknown mode')}")
     
     def generate(
         self,
@@ -471,7 +498,21 @@ class CertificateGenerator:
             return None
 
     def _create_base_certificate(self) -> Image.Image:
-        return Image.new('RGB', self.certificate_size, self.background_color)
+        """Create base certificate with improved background"""
+        # Create base image
+        certificate = Image.new('RGB', self.certificate_size, self.background_color)
+        
+        # Add subtle gradient background (optional)
+        draw = ImageDraw.Draw(certificate)
+        
+        # Add very subtle grain texture for professional look (optional)
+        width, height = certificate.size
+        for x in range(0, width, 4):
+            for y in range(0, height, 4):
+                if random.random() > 0.5:
+                    draw.point((x, y), fill=(250, 250, 250))
+        
+        return certificate
     
     def _save_certificate(self, certificate: Image.Image) -> str:
         temp_file = tempfile.NamedTemporaryFile(delete=False, suffix='.png')