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

Enhancement: Bug Details Page: 6 #2102

Merged
merged 9 commits into from
May 24, 2024

Conversation

CodeWithBishal
Copy link
Contributor

The Upper part of the page has been updated to match the design of sentry issue page, in my next PR I will update the design of the marked area
Screenshot 2024-05-18 at 4 15 12 PM

website/templates/issue3.html Fixed Show resolved Hide resolved
@DonnieBLT
Copy link
Collaborator

I would like to see the words in the buttons looks good otherwise

@CodeWithBishal
Copy link
Contributor Author

CodeWithBishal commented May 20, 2024

@DonnieBLT this is my next PR, I have added tooltip to the buttons(It shows words on hover)

Screenshot 2024-05-20 at 7 34 49 PM

@DonnieBLT
Copy link
Collaborator

We won't need tools tips you have the text on the button

@CodeWithBishal
Copy link
Contributor Author

CodeWithBishal commented May 20, 2024

@DonnieBLT Um okay, though that might mess up the design but I will get it ready by my 8th revision. This is the 6th revision, 7th is ready, as soon as you merge 6, I will create a PR for 7 then 8 with the specified changes.

Design as of 7th revision
Screenshot 2024-05-20 at 7 41 50 PM

@CodeWithBishal
Copy link
Contributor Author

@DonnieBLT

@DonnieBLT DonnieBLT enabled auto-merge (squash) May 24, 2024 14:18
@DonnieBLT DonnieBLT merged commit aad519e into OWASP-BLT:main May 24, 2024
8 checks passed
HanilJain pushed a commit to HanilJain/BLT that referenced this pull request May 31, 2024
* initial commit

* part - 1 complete

* precommit

* Update issue3.html

* precommit run

* added integrity

* Update issue3.html
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants