Правильно ли реализовано скрыть/показать блок
 

Суть проста –
При клике на элемент раскрывается блок с картой и меняется текст элемента.
При повторном нажатии закрывается блок, меняется текст элемента в исходное значение.

Вот код:

$('#show-map').click(function() {
  $('#box-map').toggle( "slow", function() {
  if($('#box-map').is(":visible")){
	$('#show-map a').html('Скрыть карту');
  }
  else{
	$('#show-map a').html('Показать на карте');
  }
  });
});

Или лучше дважды проверить: if($('#box-map').is(":visible"))

Буду рад конструктивной критике...

:write:

var show = $('#show-map'),
	box = $('#box-map'),
	text = ['Показать на карте', 'Скрыть карту'];

show.click(function() {
	box.toggle(500, function() {
		show.find('a').html(text[+box.is(':visble')]);
	});
});