nextcloud / nextcloud/mail

Buggy scroll to logic

Open
#6,187 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

1. to develop bug
Dominant language
JavaScript
Stars
1k
Forks
348
Avg merge
12h 28m
Merged PRs (30d)
91

Description

Steps to reproduce
  1. Not sure. I think I loaded the app and quickly navigated to the outbox
  2. ???
Expected behavior

No js errors logged

Actual behavior

Bildschirmfoto von 2022-04-06 19-41-07

Mail app version

1.x

Mailserver or service

No response

Operating system

No response

PHP engine version

No response

Web server

No response

Database

No response

Additional info

I'd suggest that we add some hardening to the query selector result and exit the method early if it doesn't match any elements.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the scroll-to logic and its query selector in the JavaScript frontend, then investigate the navigation path to the outbox described in the issue. Reproduce the reported JavaScript error if possible and verify that the logic handles a selector matching no elements without errors.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
frontend
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.