fixed support for optimade, poscar and some cifs; error styles

matinfio/cell/cell.ts

@@ -34,12 +34,9 @@ namespace $ {
 	}
 
 	/** 3x3 matrix to crystalline cell parameters */
-	export function $mpds_cifplayer_matinfio_cell_params_from_matrix( matrix: number[] ) {
-		const norms: number[] = []
+	export function $mpds_cifplayer_matinfio_cell_params_from_matrix( matrix: number[][] ) {
+		const norms: number[] = matrix.map( vec => math.norm( vec ) )
 		const angles = []
-		matrix.forEach( function( vec ) {
-			norms.push( math.norm( vec ) )
-		} )
 		let j = -1
 		let k = -2
 		for( let i = 0; i < 3; i++ ) {

matinfio/cif/cif.ts

@@ -83,9 +83,14 @@ namespace $ {
 				}
 				continue
 
-			} else if( fingerprt.startsWith( '_symmetry_space_group_name_h-m' ) || fingerprt.startsWith( '_space_group.patterson_name_h-m' ) ) {
+			} else if( 
+				fingerprt.startsWith( '_symmetry_space_group_name_h-m' ) 
+				|| fingerprt.startsWith( '_space_group.patterson_name_h-m' )
+				|| fingerprt.startsWith( '_space_group_name_h-m_alt' )
+			) {
 				loop_active = false
-				cur_structure.sg_name = lines[ i ].trim().substr( 31 ).replace( /"/g, '' ).replace( /'/g, '' )
+				const match = cur_line.match( /"(.*)"|'(.*)'/ )
+				cur_structure.sg_name = match?.[1] || match?.[2]
 				continue
 
 			} else if( fingerprt.startsWith( '_space_group.it_number' ) || fingerprt.startsWith( '_space_group_it_number' ) || fingerprt.startsWith( '_symmetry_int_tables_number' ) ) {

matinfio/player/player.web.ts

@@ -15,6 +15,16 @@ namespace $ {
 
 		} else {
 			cell_matrix = crystal.cell_matrix // for POSCAR and OPTIMADE
+
+			const params = $mpds_cifplayer_matinfio_cell_params_from_matrix( cell_matrix! )
+			descr = {
+				'a': params[0],
+				'b': params[1],
+				'c': params[2],
+				'alpha': params[3],
+				'beta': params[4],
+				'gamma': params[5],
+			}
 		}
 
 		if( !crystal.atoms.length ) this.$mpds_cifplayer_matinfio_log.warning( "Note: no atomic coordinates supplied" )

matinfio/spacegroup/spacegroup.web.ts

@@ -8,10 +8,10 @@ namespace $ {
 			super()
 		}
 
-		@ $mol_mem_key
-		static by_name_and_num( name: string, num: number ) {
-			const spacegroup = $mpds_cifplayer_matinfio_spacegroup.by_num( num )
-				?? $mpds_cifplayer_matinfio_spacegroup.by_name( name )
+		static by_name_or_num( name: string, num: number ) {
+			const spacegroup = num
+				? $mpds_cifplayer_matinfio_spacegroup.by_num( num )
+				: name ? $mpds_cifplayer_matinfio_spacegroup.by_name( name ) : null
 			return spacegroup ? spacegroup : $mpds_cifplayer_matinfio_spacegroup.unknown()
 		}
 

player/player.view.css

@@ -0,0 +1,7 @@
+[mpds_cifplayer_player][mol_view_error]:not([mol_view_error="Promise"]) {
+	color: var(--mol_theme_text);
+	background-image: none;
+	padding-top: 6rem;
+	align-items: flex-start;
+	justify-content: center;
+}

player/player.view.web.ts

@@ -10,8 +10,8 @@ namespace $.$$ {
 
 		@ $mol_mem
 		render(): void {
-			super.render()
 			this.structure_3d_data() //for bubbling errors into view
+			super.render()
 		}
 
 		@ $mol_mem
@@ -153,7 +153,7 @@ namespace $.$$ {
 		spacegroup() {
 			const { sg_name, ng_name } = this.structure_3d_data()
 
-			return $mpds_cifplayer_matinfio_spacegroup.by_name_and_num( sg_name, ng_name )
+			return $mpds_cifplayer_matinfio_spacegroup.by_name_or_num( sg_name, ng_name )
 		}
 
 		@ $mol_mem