Contributing.md update

Author: tanishisherewithhh

CONTRIBUTING.md

@@ -1,4 +1,68 @@
-# Contributing
+# Contributing to DynamicHUD
 
-Anyone is free to commit and make pull requests. Just make sure that you are well documenting your code and making necessary changes.
-Also please review your changes for any bugs that may occur.
+Thank you for your interest in contributing to DynamicHUD! We welcome all contributions, including bug fixes, feature suggestions, documentation updates, and new widget types or context menu skins.
+Please follow the following guidelines.
+---
+
+## **1. Getting Started**
+
+### **Setting Up the Workspace**
+DynamicHUD is built on the **Fabric Loader** toolchain using **Fabric Loom**.
+1. **Clone the repository**:
+   ```bash
+   git clone https://github.com/tanishisherewith/DynamicHUD.git
+   cd DynamicHUD
+   ```
+2. **Setup the Gradle environment**:
+   - For IntelliJ IDEA, import the folder as a Gradle project.
+   - Run the task to generate Minecraft source mappings:
+     ```bash
+     gradlew genSources
+     ```
+3. **Run Minecraft**:
+   - Run the client using the run configuration generated by Loom:
+     ```bash
+     gradlew runClient
+     ```
+
+---
+
+## **2. Coding Standards & Guidelines**
+
+### **Java Standards**
+- Use standard Java camelCase naming conventions.
+- Maintain visibility access levels. Use package-private or `protected` fields appropriately for widget extensions.
+- Do not check in raw absolute coordinates in widgets or hardcoded resolution values. Use the snap-alignment and ratio-anchored dynamic coordinate updates from `Widget.java`.
+
+### **Official Mappings**
+- DynamicHUD uses **Official Mojang Mappings** since 1.21.11 (`v4.0.0`) onwards.
+
+### **Core Components Reference**
+> More to be listed later. TODO
+- **Text & Truncation**: To prevent option names from overflowing card limits, use `Util.getTruncatedName(Component, int)` and render with `Util.drawTruncatedScaledText(...)` or `Util.drawScaledText(...)`.
+
+---
+
+## **3. Creating a Pull Request**
+
+1. **Create a Feature Branch**:
+   ```bash
+   git checkout -b feature/my-new-feature
+   ```
+2. **Write Clean Commits**:
+   - Write clear, concise commit messages.
+   - Separate refactoring from new features.
+3. **Test Your Changes**:
+   - Run a clean build to make sure java compilation passes without warnings:
+     ```bash
+     gradlew clean build
+     ```
+   - Verify that your layout strategy matches visual guides and screen resize snapping works.
+4. **Submit your PR**:
+   - Provide a clear description of the problem solved and links to relevant issues.
+   - Add screenshots or recordings if your changes affect the GUI.
+
+---
+
+## **4. Code of Conduct**
+We expect all contributors to adhere to our [Code of Conduct](CODE_OF_CONDUCT.md) in all community interactions.

src/main/java/com/tanishisherewith/dynamichud/widget/Widget.java

@@ -121,7 +121,49 @@ public float getScale() {
     public float getWidth() { return widgetBox.getWidth(); }
     public float getHeight() { return widgetBox.getHeight(); }
 
+    private Anchor determineBestAnchor(int x, int y, int width, int height, int screenWidth, int screenHeight) {
+        if (screenWidth <= 0 || screenHeight <= 0) return Anchor.TOP_LEFT;
+
+        int centerX = x + width / 2;
+        int centerY = y + height / 2;
+
+        int xZone = 0; // 0 = Left, 1 = Center, 2 = Right
+        if (centerX < screenWidth / 3) {
+            xZone = 0;
+        } else if (centerX < 2 * screenWidth / 3) {
+            xZone = 1;
+        } else {
+            xZone = 2;
+        }
+
+        int yZone = 0; // 0 = Top, 1 = Center, 2 = Bottom
+        if (centerY < screenHeight / 3) {
+            yZone = 0;
+        } else if (centerY < 2 * screenHeight / 3) {
+            yZone = 1;
+        } else {
+            yZone = 2;
+        }
+
+        if (xZone == 0) {
+            if (yZone == 0) return Anchor.TOP_LEFT;
+            if (yZone == 1) return Anchor.CENTER_LEFT;
+            return Anchor.BOTTOM_LEFT;
+        } else if (xZone == 1) {
+            if (yZone == 0) return Anchor.TOP_CENTER;
+            if (yZone == 1) return Anchor.CENTER;
+            return Anchor.BOTTOM_CENTER;
+        } else {
+            if (yZone == 0) return Anchor.TOP_RIGHT;
+            if (yZone == 1) return Anchor.CENTER_RIGHT;
+            return Anchor.BOTTOM_RIGHT;
+        }
+    }
+
     private void calculateOffset(int initialX, int initialY, int screenWidth, int screenHeight) {
+        if (screenWidth > 0 && screenHeight > 0) {
+            this.anchor = determineBestAnchor(initialX, initialY, (int) getWidth(), (int) getHeight(), screenWidth, screenHeight);
+        }
         int anchorX = anchor.getBaseX(screenWidth);
         int anchorY = anchor.getBaseY(screenHeight);
         this.offsetX = initialX - anchorX;
@@ -130,15 +172,12 @@ private void calculateOffset(int initialX, int initialY, int screenWidth, int sc
 
     // Update position based on anchor and offset
     void updatePosition(int screenWidth, int screenHeight) {
-        if (offsetX == 0 && offsetY == 0) {
-            calculateOffset(x, y, mc.getWindow().getGuiScaledWidth(), mc.getWindow().getGuiScaledHeight());
-        }
-
         int anchorX = anchor.getBaseX(screenWidth);
         int anchorY = anchor.getBaseY(screenHeight);
         this.x = anchorX + offsetX;
         this.y = anchorY + offsetY;
         clampPosition();
+        calculateOffset(x, y, screenWidth, screenHeight);
     }
 
     public void setPosition(int x, int y) {
@@ -220,7 +259,9 @@ private void renderWidgetInEditor(GuiGraphics graphics, int mouseX, int mouseY)
     public boolean mouseClicked(double mouseX, double mouseY, int button) {
         if (widgetBox.isMouseOver(mouseX, mouseY) && button == GLFW.GLFW_MOUSE_BUTTON_LEFT) {
             wasDragged = false;
-            if (!isLocked) {
+            if (isLocked) {
+                toggle(); // Static widgets toggle immediately
+            } else {
                 startX = (int) (mouseX - x);
                 startY = (int) (mouseY - y);
                 dragging = true;
@@ -232,8 +273,6 @@ public boolean mouseClicked(double mouseX, double mouseY, int button) {
                         }
                     }
                 }
-            } else {
-                toggle(); // Static widgets toggle immediately
             }
             return true;
         }