Merge pull request #41 from ccswbs/tq-iss40

Fix phone extension logic in Contact component

Author: tqureshi-uog

.changeset/all-ads-walk.md

@@ -0,0 +1,5 @@
+---
+'@uoguelph/react-components': patch
+---
+
+Refactor phone extension handling

.changeset/pre.json

@@ -0,0 +1,13 @@
+{
+  "mode": "pre",
+  "tag": "rc",
+  "initialVersions": {
+    "@uoguelph/react-components": "1.7.0",
+    "@uoguelph/storybook": "1.0.1",
+    "@uoguelph/tailwind-theme": "1.0.2",
+    "@uoguelph/web-components": "2.0.10"
+  },
+  "changesets": [
+    "all-ads-walk"
+  ]
+}

package-lock.json

@@ -1,5 +1,11 @@
 # @uoguelph/react-components
 
+## 1.7.1-rc.0
+
+### Patch Changes
+
+- 52d524f: Refactor phone extension handling
+
 ## 1.7.0
 
 ### Minor Changes

packages/react-components/CHANGELOG.md

@@ -1,6 +1,6 @@
 {
   "name": "@uoguelph/react-components",
-  "version": "1.7.0",
+  "version": "1.7.1-rc.0",
   "description": "University of Guelph React Components Library",
   "type": "module",
   "scripts": {

packages/react-components/package.json

@@ -26,10 +26,9 @@ export function ContactPhone({ number, extension, className }: ContactPhoneProps
   return (
     <div className={`uofg-contact-phone-container ${twMerge(container(), className)}`}>
       <FontAwesomeIcon icon={faPhone} className="uofg-contact-phone-icon" />
-      <a className={`uofg-contact-phone ${link()}`} href={`tel:${number}`}>
-        {number}
+      <a className={`uofg-contact-phone ${link()}`} href={`tel:${number}${extension ? `;${extension}` : ''}`}>
+        {number}{extension && <span className="uofg-contact-phone-extension"> - Ext. {extension}</span>}
       </a>
-      {extension && <span className="uofg-contact-phone-extension"> - Ext. {extension}</span>}
     </div>
   );
 }