Skip to content

feat: Make the "report a bug" links more visible - #3883

Merged
rjsparks merged 3 commits into
ietf-tools:mainfrom
larseggert:fix-report-a-bug
Apr 26, 2022
Merged

feat: Make the "report a bug" links more visible#3883
rjsparks merged 3 commits into
ietf-tools:mainfrom
larseggert:fix-report-a-bug

Conversation

@larseggert

Copy link
Copy Markdown
Collaborator

Also rework the page footer somewhat, while we're here.

Also rework the page footer somewhat, while we're here.
@rjsparks

Copy link
Copy Markdown
Member

I'm not sure this is enough.
(I can live with the suggested footer rework).
I think we may need (for awhile) to put report-a-bug in the menus somewhere, or at least on the menu bar

@larseggert

Copy link
Copy Markdown
Collaborator Author

It's in the "other" menu

@rjsparks

Copy link
Copy Markdown
Member

and yet we still have chairs asking "how do I report a bug?" regularly.

I'm thinking of something more like:

image

unless you think that's just too obnoxious.

@rjsparks

Copy link
Copy Markdown
Member

If we go that route, we could place it only for logged in users to start with

@codecov

codecov Bot commented Apr 26, 2022

Copy link
Copy Markdown

Codecov Report

Merging #3883 (8392733) into main (92e67ac) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #3883   +/-   ##
=======================================
  Coverage   87.99%   87.99%           
=======================================
  Files         297      297           
  Lines       38771    38771           
=======================================
+ Hits        34115    34118    +3     
+ Misses       4656     4653    -3     
Impacted Files Coverage Δ
ietf/nomcom/utils.py 92.11% <0.00%> (+0.54%) ⬆️
ietf/utils/pipe.py 87.50% <0.00%> (+4.16%) ⬆️

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 0dec52f...8392733. Read the comment docs.

@rjsparks

Copy link
Copy Markdown
Member

I'll merge this, and move the conversation about doing more elsewhere.

@rjsparks
rjsparks merged commit 53a2b61 into ietf-tools:main Apr 26, 2022
@larseggert
larseggert deleted the fix-report-a-bug branch April 26, 2022 17:35
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jun 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants