Skip to content

OBPIH-6780 Add Page numbers on Picklist and Delivery Note#5199

Merged
awalkowiak merged 3 commits intodevelopfrom
OBPIH-6780
Apr 14, 2025
Merged

OBPIH-6780 Add Page numbers on Picklist and Delivery Note#5199
awalkowiak merged 3 commits intodevelopfrom
OBPIH-6780

Conversation

@SebastianLib
Copy link
Collaborator

✨ Description of Change

Link to GitHub issue or Jira ticket:

Description:


📷 Screenshots & Recordings (optional)

@github-actions github-actions bot added the domain: frontend Changes or discussions relating to the frontend UI label Apr 10, 2025
size: letter;
background: white;
@top-center { content: element(header) }
@bottom-center { content: element(footer) }
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Actually, this element doesn’t even exist in the file, so I could remove the style for div.footer. But since a lot of the other unused styles haven’t been removed either, I’m not going to remove this one for now. Or maybe should I?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Remind me, why you could not re-add the:

<div class="footer">
    <div id="page-footer" class="small">
        <div>
            Page <span class="page" /> of <span class="pagecount" />
        </div>
    </div>
</div>

We have all the styling there, so perhaps this could work as well?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Ok then i think you can get rid of the unused styling

@codecov
Copy link

codecov bot commented Apr 10, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 8.12%. Comparing base (3327c0e) to head (05eaee0).
Report is 145 commits behind head on develop.

Additional details and impacted files
@@            Coverage Diff            @@
##             develop   #5199   +/-   ##
=========================================
  Coverage       8.12%   8.12%           
  Complexity       939     939           
=========================================
  Files            638     638           
  Lines          43211   43211           
  Branches       10503   10503           
=========================================
  Hits            3512    3512           
  Misses         39163   39163           
  Partials         536     536           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@awalkowiak awalkowiak merged commit b6e39db into develop Apr 14, 2025
9 checks passed
@awalkowiak awalkowiak deleted the OBPIH-6780 branch April 14, 2025 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain: frontend Changes or discussions relating to the frontend UI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants