Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow clicking on either label or notation in hierarchy #1211

Merged
merged 1 commit into from
Sep 16, 2021

Conversation

osma
Copy link
Member

@osma osma commented Sep 16, 2021

Fixes #1210

Clicking on notation codes in the hierarchy didn't work in the previous release, and in current master even clicking on labels is broken. This PR fixes both issues. The problem was that when clickin on a <span> element, the URL for the target concept/page wasn't extracted properly.

@osma osma added the bug label Sep 16, 2021
@osma osma added this to the 2.12 milestone Sep 16, 2021
@osma osma self-assigned this Sep 16, 2021
@sonarcloud
Copy link

sonarcloud bot commented Sep 16, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@codecov
Copy link

codecov bot commented Sep 16, 2021

Codecov Report

Merging #1211 (6242fa2) into master (9db5409) will increase coverage by 0.08%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1211      +/-   ##
============================================
+ Coverage     68.04%   68.13%   +0.08%     
  Complexity     1619     1619              
============================================
  Files            32       32              
  Lines          3962     3973      +11     
============================================
+ Hits           2696     2707      +11     
  Misses         1266     1266              
Impacted Files Coverage Δ
model/sparql/GenericSparql.php 92.14% <0.00%> (+0.08%) ⬆️
model/sparql/JenaTextSparql.php 92.00% <0.00%> (+0.16%) ⬆️
model/resolver/WDQSResource.php 94.44% <0.00%> (+0.32%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9db5409...6242fa2. Read the comment docs.

@osma osma merged commit c960e09 into master Sep 16, 2021
@osma osma deleted the issue1210-fix-hierarchy-click-on-notation branch September 16, 2021 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clicking on notation code in hierarchy loads wrong page
1 participant