Skip to content

fix popup issue#63

Merged
XploY04 merged 6 commits into
pointblank-club:stagingfrom
Hemang360:popupfix
Aug 21, 2025
Merged

fix popup issue#63
XploY04 merged 6 commits into
pointblank-club:stagingfrom
Hemang360:popupfix

Conversation

@Hemang360

Copy link
Copy Markdown
Contributor

Summary: What does this PR do?

Fix popup issue due to which resumes were not opening in safari.

Which issue(s) this PR fixes:

Changes Made

Describe the changes you've made in this PR:

  • Feature implementation
  • Bug fix
  • Documentation update
  • Code refactoring
  • Configuration changes
  • Other (please specify)

Type of Change

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Documentation update
  • Performance improvement
  • Code cleanup or refactor

How Has This Been Tested?

Describe the tests you ran:

  • Unit tests
  • Integration tests
  • Manual tests
  • Other (please specify)

Please describe the test cases and expected behavior:
1.
2.

Screenshots (if applicable)

Dependencies

Documentation

  • I have updated the documentation accordingly
  • Documentation update is not required

Comments:

Reviewer Notes

@github-actions github-actions Bot added Frontend Auto-generated label for Frontend changes size-XS Auto-generated label for size-XS changes labels Aug 6, 2025

@XploY04 XploY04 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of opening it on the same tab, make a component which pops up on view resume

@govindup63

Copy link
Copy Markdown
Collaborator

damn I was going to approve this

@github-actions github-actions Bot added size-M Auto-generated label for size-M changes and removed size-XS Auto-generated label for size-XS changes labels Aug 17, 2025
@github-actions github-actions Bot added size-M Auto-generated label for size-M changes and removed size-M Auto-generated label for size-M changes labels Aug 17, 2025
@github-actions github-actions Bot added size-M Auto-generated label for size-M changes and removed size-M Auto-generated label for size-M changes labels Aug 19, 2025
@XploY04 XploY04 merged commit 2f6f215 into pointblank-club:staging Aug 21, 2025
2 checks passed
vanishajr pushed a commit to vanishajr/pb-placements that referenced this pull request Aug 21, 2025
* fix popup issue

* component added

* fix popup issue

* component added
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Frontend Auto-generated label for Frontend changes size-M Auto-generated label for size-M changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants