search

wurmlab / sequenceserver

Intuitive graphical web interface for running BLAST bioinformatics tool (i.e. have your own custom NCBI BLAST site!)
https://sequenceserver.com
GNU Affero General Public License v3.0
271 stars 113 forks source link

Resolved Sidebar Clickability Issue for Non-Interactive Elements #714

Closed gorsyan closed 9 months ago

gorsyan commented 10 months ago

Fixed Bug #712 Corrected an Issue Where the Sidebar Element Remained Clickable Despite Being Intended as Non-Interactive, Leading to User Data Loss. Implemented Explicit CSS Restrictions to Disable Pointer Events on Elements with the 'Disabled' Attribute. Additionally, Added a Check to Ensure Event Listeners in the Sidebar Element are Inactive if the Array of Selected Items is Zero, Preventing Unintended Interactions.

ghost commented 10 months ago
👇 Click on the image for a new way to code review #### [![Review these changes using an interactive CodeSee Map](https://s3.us-east-2.amazonaws.com/maps.codesee.io/images/github/wurmlab/sequenceserver/714/9e684a76/a67bfddcd9a93c72934493af6fc06593706e74ed.svg)](https://app.codesee.io/r/reviews?pr=714&src=https%3A%2F%2Fgithub.com%2Fwurmlab%2Fsequenceserver) #### Legend CodeSee Map legend
yannickwurm commented 9 months ago

Hiya @gorsyan - I'm sorry - I'm pretty sure that this may now need to be rebased given that there have been minor changes to what contributes to the .min.js

codeclimate[bot] commented 9 months ago

Code Climate has analyzed commit a67bfddc and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 1

The test coverage on the diff in this pull request is 100.0% (80% is the threshold).

This pull request will bring the total coverage in the repository to 81.9% (0.0% change).

View more on Code Climate.