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

Push buttons to bottom in Send and Receive Screens #1672

Merged
merged 3 commits into from
Oct 27, 2022
Merged

Conversation

im-adithya
Copy link
Member

Describe the changes you have made in this PR

Title

Link this PR to an issue [optional]

Fixes #1671
Fixes #1670
Fixes #1650

Type of change

  • fix: Bug fix (non-breaking change which fixes an issue)

Screenshots of the changes [optional]

Screenshot 2022-10-26 at 3 16 46 PM

How has this been tested?

Manually

Checklist

  • My code follows the style guidelines of this project and performed a self-review of my own code
  • New and existing tests pass locally with my changes
  • I checked if I need to make corresponding changes to the documentation (and made those changes if needed)

@github-actions
Copy link

github-actions bot commented Oct 26, 2022

🚀 Thanks for the pull request!

Here are the current build files for testing:

Download and unzip the file for your browser. Refer to the readme for detailed install instructions.


This build is brought to you by: Adam Fiscor (who recently dropped 1337 sats):

The future is bright. We just have a lot of work to do.

Want to sponsor the next build? send some sats to ⚡️[email protected] (don't forget to provide your name)

Don't forget: keep earning sats!

Copy link
Contributor

@escapedcat escapedcat left a comment

Choose a reason for hiding this comment

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

ack

@@ -191,7 +191,7 @@ function Receive() {
}

return (
<div>
<div className="h-full flex flex-col overflow-y-auto no-scrollbar">
Copy link
Collaborator

Choose a reason for hiding this comment

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

are we save to not want scrollbars?

Copy link
Contributor

Choose a reason for hiding this comment

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

We decided against scrollbars a long time ago. We already do this on most screens.
If we want to change this we should do it in a new issue. Or is this blocking?

Copy link
Collaborator

Choose a reason for hiding this comment

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

ok

Copy link
Contributor

Choose a reason for hiding this comment

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

I think we're currently trying to fit everything into the extension-screen without scrollbars:
#1662

@bumi bumi merged commit 50c5f3b into master Oct 27, 2022
@bumi bumi deleted the task-send-receive branch October 27, 2022 09:44
bumi added a commit that referenced this pull request Nov 1, 2022
…nnector

# By Lisa Oppermann (15) and others
# Via GitHub (22) and others
* feature/generic-webln-request: (58 commits)
  feat: add generic request method to webln
  fix: publisher link on AllowanceView
  Translated using Weblate (Italian)
  Update all development Yarn dependencies (2022-10-29)
  fix: align reject/close function params
  fix: navigate on send
  Update dayjs to version 1.11.6
  fix: disable input fields on Create Invoice #1659 (#1668)
  Update react-i18next to version 12.0.0
  Push buttons to bottom in Send and Receive Screens (#1672)
  refactor(lnurl): getDetails - avoid typecasting (#1545)
  style: prettier
  Translated using Weblate (Italian)
  Update tailwindcss to version 3.2.1
  style: prettier
  Translated using Weblate (Portuguese (Brazil))
  style: prettier
  Translated using Weblate (Italian)
  Translated using Weblate (Swedish)
  fix: remove response and additional error message
  ...

# Conflicts:
#	yarn.lock
bumi added a commit that referenced this pull request Nov 1, 2022
# By Lisa Oppermann (15) and others
# Via GitHub (22) and others
* master: (57 commits)
  fix: publisher link on AllowanceView
  Translated using Weblate (Italian)
  Update all development Yarn dependencies (2022-10-29)
  fix: align reject/close function params
  fix: navigate on send
  Update dayjs to version 1.11.6
  fix: disable input fields on Create Invoice #1659 (#1668)
  Update react-i18next to version 12.0.0
  Push buttons to bottom in Send and Receive Screens (#1672)
  refactor(lnurl): getDetails - avoid typecasting (#1545)
  style: prettier
  Translated using Weblate (Italian)
  Update tailwindcss to version 3.2.1
  style: prettier
  Translated using Weblate (Portuguese (Brazil))
  style: prettier
  Translated using Weblate (Italian)
  Translated using Weblate (Swedish)
  fix: remove response and additional error message
  fix: use math floor everywhere
  ...

# Conflicts:
#	yarn.lock
bumi added a commit that referenced this pull request Nov 5, 2022
# By Lisa Oppermann (15) and others
# Via GitHub (35) and escapedcat (1)
* master: (75 commits)
  fix: remove runcitadel sdk dependency (#1681)
  chore(deps): update puppeteer to v19
  revert(deps): puppeteer and fake-indexdb
  Update fake-indexeddb to version 4.0.0
  Update @types/chrome to version 0.0.200
  Update puppeteer to version 19.2.0
  fix: publisher link on AllowanceView
  Translated using Weblate (Italian)
  Update all development Yarn dependencies (2022-10-29)
  chore: remove showcases
  chore: remove topup app
  fix: align reject/close function params
  fix: navigate on send
  Update dayjs to version 1.11.6
  fix: disable input fields on Create Invoice #1659 (#1668)
  Update react-i18next to version 12.0.0
  Push buttons to bottom in Send and Receive Screens (#1672)
  refactor(lnurl): getDetails - avoid typecasting (#1545)
  style: prettier
  Translated using Weblate (Italian)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants