[openstreetmap/openstreetmap-website] Added indicator message preview is empty. (PR #4894)

Anton Khorev notifications at github.com
Sun Jun 30 23:40:52 UTC 2024


@AntonKhorev commented on this pull request.



> +  /*
+   * Install a click handler to return 'required' attribute for
+   * message_title (Subject) when send button is pressed.
+   */
+  $(".btn-primary[id=\"send_message_button_id\"]").click(function () {
+    var messageTitleElement = document.getElementById("message_title");
+    if (messageTitleElement) {
+      messageTitleElement.setAttribute("required", "required");
+    }
+  });
+
+  /*
+   * Install a click handler to return 'required' attribute to
+   * diary_entry_title (Subject) element, .. when publish / update
+   * button is pressed.
+   */
+  $(".btn-primary[id=\"publish_or_update_diary_entry_button_id\"]").click(function () {
+    var diaryEntryTitleElement = document.getElementById("diary_entry_title");
+    if (diaryEntryTitleElement) {
+      diaryEntryTitleElement.setAttribute("required", "required");
+    }
+  });
+

None of this is required for what you're trying to do.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4894#pullrequestreview-2150219688
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/4894/review/2150219688 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240630/2265b9bc/attachment.htm>


More information about the rails-dev mailing list