kosmos/akkounts
kosmos/akkounts
8
1
Fork 0
Code Issues 19 Pull Requests 1 Actions Packages Releases 12 Activity

Various UI improvements #43

Merged
raucao merged 5 commits from ui/misc into master 2021-11-24 02:41:55 +00:00
Conversation 2 Commits 5 Files Changed 4 +2 -2
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 12a9d4674b - Show all commits

4
app/views/wallet/index.html.erb
View File

@ -8,10 +8,10 @@
<div class="md:col-span-4 mt-4 md:mt-0"> <div class="md:col-span-4 mt-4 md:mt-0">
<p class="font-mono md:text-right p-4 bg-gray-50 shadow rounded-lg overflow-hidden"> <p class="font-mono md:text-right p-4 bg-gray-50 shadow rounded-lg overflow-hidden">
<% if @balance %> <% if @balance %>
<span class="text-xl">48590<%= @balance %> sats</span><br> <span class="text-xl"><%= @balance %> sats</span><br>
raucao marked this conversation as resolved Outdated
galfert commented 2021-11-24 02:22:59 +00:00
Outdated
Review
Copy Link

There is a hardcoded number in front of the actual balance.

There is a hardcoded number in front of the actual balance.
<span class="text-sm text-gray-500">Available balance</span> <span class="text-sm text-gray-500">Available balance</span>
<% else %> <% else %>
<span class="text-xl">n/a<%= @balance %> sats</span><br> <span class="text-xl">n/a sats</span><br>
raucao marked this conversation as resolved Outdated
galfert commented 2021-11-24 02:22:12 +00:00
Outdated
Review
Copy Link

Looks like a copy/paste error. The <%= @balance %> should be removed.

Looks like a copy/paste error. The `<%= @balance %>` should be removed.
<span class="text-sm text-gray-500">Balance unavailable</span> <span class="text-sm text-gray-500">Balance unavailable</span>
<% end %> <% end %>
</p> </p>