Merge pull request #48 from alecrem/feature/issue-44/link-legal-cards-on-list-check

pages/1_Check_Card_List.py

@@ -55,4 +55,7 @@ if "English" in input_cards and "日本語" in input_cards:
         hide_index=True,
     )
 
+if input_cards.shape[0] > 0:
+    input_cards[input_cards["Legal"] == "✅"].apply(lib.row_to_button_link, axis=1)
+
 streamlit_common.footer.write_footer()

streamlit_common/lib.py

@@ -14,7 +14,15 @@ def row_to_link(row: pd.DataFrame) -> None:
     cardname = row["name"]
     if row.name_ja is not "":
         cardname = f"{cardname} / {row.name_ja}"
-    st.markdown(f"- [{cardname}]({row.link})")
+    st.link_button(cardname, row.link)
+
+
+def row_to_button_link(row: pd.DataFrame) -> None:
+    """Prints a list item with a Scryfall link for the card in the row passed"""
+    cardname = row.English
+    if row.日本語 is not "":
+        cardname = f"{cardname} / {row.日本語}"
+    st.link_button(cardname, compose_scryfall_url(row.English))
 
 
 def get_legal_cardnames(cardname: str, mslist_df: pd.DataFrame) -> list: