북마크 모달 바깥을 클릭해도 북마크가 생성됩니다

This might be me but I am used to closing a modal I want to cancel by clicking outside of the modal.

When creating a bookmark, the modal does disappear when you click outside of it, however the bookmark is still being created or changed. This is not what I expected to happen.

Clicking the X or pressing ESC to close the modal does actually cancel the action.

firefox_Bu7E734mqK

9개의 좋아요

I have just repro’d this on your post. :slightly_smiling_face:

Clicking ‘off-screen’ does indeed create the bookmark rather than exiting without saving.

6개의 좋아요

I’m not sure, this could be the intended behaviour in line with how browsers (at least firefox iirc) treats bookmarks.

You click the bookmark button and it creates the bookmark unless you specifically ask not to.

3개의 좋아요

But since the dawn of time closing a modal performs no action.

4개의 좋아요

I agree this seems like a bug. I am able to repro here as well. Clicking outside the modal creates a bookmark with no reminder set.

5개의 좋아요

이 기능 개발 시 의도적으로 이렇게 설계되었습니다. 아이디어는 Firefox의 북마크 모달을 참고한 것입니다. 로직은 다음과 같습니다:

  • 새 북마크를 클릭한 후 모달 바깥을 클릭하면, 모달이 닫히고 북마크가 생성됩니다.
  • 새 북마크를 클릭한 후 취소 버튼을 클릭하면, 북마크가 생성되지 않습니다.
  • 기존 북마크를 클릭하면 새로운 “북마크 제거” 버튼이 나타나며, 이 버튼을 사용하여 북마크를 제거할 수 있습니다.

이것이 혼란스럽다면 변경해야 한다는 주장은 아닙니다. 하지만 기술적으로는 버그가 아닙니다. 이 이슈를 #contribute:ux로 옮기겠습니다. 전체 페이지 모달(Discourse)과 플라이아웃 메뉴(Firefox) 사이의 차이가 동작 방식을 변경할 만큼 중요한 차이일 수 있을까요?

8개의 좋아요

I guess my main problem is that it is not consistent with other modals in Discourse. Clicking outside the Flag modal does not create a flag, clicking outside the Share modal does not share the post.

8개의 좋아요

I’m going to guess that there never used to be a popup/modal at all, so it made sense at the time when it was added to keep the old functionality (ie you’ve clicked the bookmark button so you get a bookmark). But overall it does seem odd. Having said that, I bookmark things all the time and had never noticed this quirk!

(The way browsers automatically add bookmarks when I accidentally hit control-D has annoyed me for years…)

1개의 좋아요

Yeah, to split hairs we can say that the bookmark button click is the action, because everything in the modal following is optional. Sharing and Flagging require an additional action or more information and there’s nothing for us to do with a single click. In hindsight that’s probably a lot harder to figure out than consistent modal behavior.

One thing that could maybe clear it up is a different form of menu… this doesn’t work perfectly as shown in this quick mock-up, but if we want to keep the single-click bookmarks it could be worth exploring. This way you could more clearly see the button change state when clicked, and it’s not a full modal, so maybe that makes the click-away behavior less confusing.

6개의 좋아요

Not a bad idea,

Please note that right now the bookmark is actually added at the moment the modal is dismissed, not at the moment it is opened. So that would require either a heavy refactoring or faking the button state.

2개의 좋아요

Going to mention @martin here, I do think this is intentional, but I also understand the confusion.

In real world usage this “over bookmarking” almost never bites me, but I can get the suprise here.

5개의 좋아요

Not a bug, this is intentional, and it has been from the very beginning of bookmarks. This is to remove friction from creating bookmarks and as Kris said:

Maybe I could see this as a user preference or something, but it has been this way for at least 2 years now without issue so I’d like to have more reports of it being a problem before we act on it.

7개의 좋아요

once agai, this is another issue I was told to come here about. it’s copied from that forum, you can get the get the link here

When pressing F on a topic, the Bookmark modal appears. My first reaction is to click off because I probably didn’t mean to open it, because a normal F is probably too easy to open it in the first place.

But, that saves it as a bookmark.

This isn’t a bug because the same thing happens most forums, it’s just an annoyance.

What I think should happen is when I click off, it does nothing and ignores the bookmark. But, if I click the “Save” button, THEN it can make the bookmark.

In summary, just make it so clicking off does nothing, and you have to click Save before a bookmark is saved.

5개의 좋아요

That’s different, you’re clicking a button that’s “Yeah, I wanna bookmark this.”
On Discourse, you probably pressed F on accident and dont want to bookmark it.

As OP says it’s not consistent with all of the other modals in discourse, I think it should be changed.

1개의 좋아요