3.0.0 Bug Report Thread

Bug Reports
  • V3.1.4 using Persona

    • chat notification remains despite reading the last PM (this has been an issue since V3.x)

    Deeper problems

    Replies/Composer:

    • dropdown-toggle is not displaying so cannot access options

    • cannot launch composer when hitting reply on user post or clicking floating post draft icon

    • need to refresh the page to see a reply or "quick reply" appear in topic

    • neither @user-name ":" emoji menu pop-up is appearing when in quick-reply box

    ACP:

    • Appearance > Themes : Hangs on "Checking for installed themes..."

    Browser: Safari 16.6

  • @omega I couldn't repro any of these on persona, I was on latest develop branch with persona 13.0.73.

    Are you seeing any errors in the browser console? Most of these sound like there is a connection issue.

  • @baris Yea loads. I will check cloud flare I've probably screwed something up, but no changes have been made other than the update and then these problems started in earnest, i.e. no configuration changes were made, that's not to say the configuration is not great either!

    One example of many errors when I click on the toggle-dropdown post menu

    [Error] Failed to load resource: the server responded with a status of 403 (Forbidden) (socket.io, line 0)...https://www/socket.io/?EIO=4&transport=polling&t=OYBO720

    During the upgrade there was an error, that appeared at the end but I'v lost a copy of that, it didn't seem fatal to the overall installation, as nodeBB restarted.

    I'm seeing this thread and it looks very similar, so I'll check that out too https://community.nodebb.org/topic/17284/err-of-update-from-2-8-11-to-3-1-4/11

  • @baris This is what fixed ma' problem https://community.nodebb.org/post/93738

    I ran the upgrade again and everything is as it should be as far as I can see.

  • @baris Is I suppose to see a checkbox ☑ in every listed post as an admin or moderator, i.e. is always on in the UI for these users types?

  • Can you share a screenshot? Posts aren't supposed to have a checkbox AFAIK.

  • @baris @omega this happens on my forum also

    b64f7335-740c-403d-b39f-5986d9bf4421-image.png

  • Oh that's by design, harmony let's you select topics with those checkboxes.

  • @baris ...ah, "by design" can it be un-designed 😉

    I can see what you are doing here, or it looks like you are trying to make something very clear to avoid confusion, "how does one select posts" to apply topic tools action to etc. etc.

    The old click the avatar confusion of the past (which I found rather a neat approach once you knew it) iirc there might have been some topic on that recently I vaguely saw, anyhooo... usually the approach is to toggle those visible when say "topic tools" is selected i.e. activated, the posts are then selected and then the desired action selected next and applied.

    I thought I was in the "activated" stage of that procedure, not in passive browsing mode.

    Is I the only one to think this is outside normal convention in general?

  • @omega the decision to separate the avatar and checkboxes was mine. It was done because clicking an avatar should always send you to a user profile. To turn it into a checkbox for some users and not others is confusing.

  • @julian Yea, I'm not lamenting that loss but I liked it, cause I'm odd that way, but conventions eh.

    However now that you said that, I was thinking most normal users or the rally 99.9% of a userbase, would never be selecting posts in the first instance (usually) only Mods or Admins would have to encounter the outside-convention approach of the past.

    I still like / prefer the idea of being in a active mod role or not. Such a clear UI delineation of being in active mod role with mod powers versus just browsing is a nice feedback touch, when things are busy.

    As an idea-example, when in a mods-tools-activated-mode, the post listing (or post) might get a obvious grey outline (or overlay to make it dull but legible, monochrome even) and when selected this might change to a brighten up to normal or get a stronger colour outline, red or bright blue, yellow.

    So scrolling up and down all the sleeted post are visually clear and strong to the mod/admin.

    Each listing is like selecting as if it's an entire card.

    Modding in the past on other platforms was always a precarious endeavour, and when ti went wrong, oh dear, as much as checkbox is old school thinking in terms of UI, i may work but consider the above if it's deemed useful.

  • Right I can see some of the UI behaviour I suggested is how it works in terms of visual feedback when selecting, fine, I hadn't looked at it deeply, so ignore all that I wrote on those points.

    Ultimately, something if feasible other than the checkbox would be visually less troublesome meaning the same visual styling can be consistent for all users, as I assume normal users don't see these checkboxes as they have no need or access to "topic tools" or do they?

  • @omega they do in the /recent and /unread pages, as there's sometimes need to select multiple topics.

  • @julian

    Ok, hmmm, well then it's even more an issue since the always-on checkbox is killing the layout n'look.

    It's either stick with the checkbox as is or consider select listings as cards option instead, but have a a toggle on/off state, so no matter whatever the selection method is it's hidden for general browsing, and only appears when activated (needed).

    If sticking with checkbox, consider aligning it to the left of the avatar not below, first in row, which is the logical or conventional place to put it IMHO, and because it's also adding to the height of each listing afaict, it's kinda wedged in there as is tbh.

    It behaves similar in discourse like this, and funnily enough it took me some time to find that button to turn on the column, so I'd say in a toggle on/ff scenario, have that topic tool button display an on/off state in the nav bar, i.e. but no buttons anywhere else. Combine the function into what is already un the UI.

    I like the topic tools counter, it's a nice visual feed back to have, however one thing I'm not seeing is a "select all" option, have I missed this?

  • Enable Live Reload (3.1.4)

    I've detect the "Enabled Live Reload" is not behaving correctly, while it refreshes the page after an update (or appears to), it doesn't actually take affect last changes, and a forced browser page reload is required to see the latest results, it's like the reload is one change behind.

  • Navbar - fit and finish, bug.

    Yea visual bug is a bug, to borrow some common coding parlance. So if something is visually buggin' me, it's a bug! 🙂

    Since I was playing around with this in the live editor it caused me to notice the below here on community:

    nav-bar-fit-finish.png

    Perhaps you're testing something out but when I came across on my own deployment, I moved to "close the gap" or at least some of the gaps, among other things.

    You can test with this quickly with the live custom code, which also applies some increased contrast with some drop shadow TLC and I think this works universally:

    .sticky-tools {
         background-color: rgba(0, 0, 0, 0.0);
         padding: 0rem 0;
    }
    
    .navbar {
        background-color: white;
        box-shadow: 0 4px 8px 0 rgba(0, 0, 0, 0.2), 0 6px 20px 0 rgba(0, 0, 0, 0.19);
    }
    

    I'm on a mission to "close the gap" on desktop and mobile, that gap is a bug, a fit and finish bug! Don't be fooled! 👷

  • One comment on the Data-line-number box looking very nice indeed with a Copy & Paste button perfect click and response feedback 👌

  • Screen goes entirely black:

    Reply -> Zen Mode -> Hide

    (Type Esc to exit the black screen)

    I reproduced this in Chrome, Firefox, and Brave. I am using v3.1.4, but it is also reproducible on this forum.

  • I found today that if in admin>settings>reputation you try and change any of the values, it doesnt let them save because it objects to other values being less than 0.
    For example this was an existing setting
    Minimum reputation to send chat messages
    -100

    It flags that up (cant be less than 0) when I try to save the other value I changed
    I dont want to make that zero, otherwise someone who is at -1 will be stopped from sending a chat message.
    it was ok at negative before, so I think its a bug that it doesnt allow it now?

  • The problem is more widespread, cant set any value less than zero.
    Im trying to figure out when this change crept in. It doesnt make sense?

    Zero.jpg


Suggested Topics


  • 3 Votes
    25 Posts
    858 Views

    @baris Awesome, thank you!

  • 0 Votes
    13 Posts
    6k Views

    i also added this code to load default preferences (inside the "success" function when creating a new user)

    User.setSetting(uid, 'dailyDigestFreq', 'week', null); User.setSetting(uid, 'notificationSounds', 1, null); User.setSetting(uid, 'followTopicsOnCreate', 1, null); User.setSetting(uid, 'followTopicsOnReply', 1, null); User.setSetting(uid, 'sendChatNotifications', 1, null);
  • 0 Votes
    3 Posts
    3k Views

    You know that if you edit a file of NodeBB itself it may refuse to update by git pull because a merge would be needed?
    So it may be that you're working with an old version of NodeBB o_O

    Anyways...
    If you are able to delete a post you have the permission to also restore it. Because of this the post doesn't get removed but only gets less opacity (within the official themes).
    I just don't know in which case the post could be removed from DOM after reload, may be an inofficial theme or an old version of NodeBB (maybe because of the reason above).

  • 0 Votes
    7 Posts
    2k Views

    It worked!!!

    git pull -> nodebb upgrade -> start

    Thank you SO MUCH!

  • 0 Votes
    8 Posts
    3k Views

    @BarveyHirdman said:

    Not sure node why app --setup wants to use ./src/emailer.js

    That file is required during admin user creation because we send the "welcome to NodeBB" email during the user creation process.

    A little redundant, as third-party emailers aren't even installed or setup yet during installation, but that's specifically the reason why it is being require'd